Tapestry 5
  1. Tapestry 5
  2. TAP5-1637

Tapestry should use SoftReferences for page instances, rather than run a janitor thread to clean them up

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.3
    • Fix Version/s: 5.3
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      From the discussion on the mailing list; this would let the VM manage the memory better than Tapestry can.

      Some newly introduced configuration symbols will go away.

        Issue Links

          Activity

          Howard M. Lewis Ship created issue -
          Howard M. Lewis Ship made changes -
          Field Original Value New Value
          Assignee Howard M. Lewis Ship [ hlship ]
          Howard M. Lewis Ship made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Fix Version/s 5.3 [ 12316024 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in tapestry-trunk-freestyle #509 (See https://builds.apache.org/job/tapestry-trunk-freestyle/509/)
          TAP5-1637: Use SoftReferences to cache Page instances, rather than an explicit janitor job

          hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1166889
          Files :

          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/SymbolConstants.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PageSourceImpl.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/services/TapestryModule.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/services/AppModule.java
          Show
          Hudson added a comment - Integrated in tapestry-trunk-freestyle #509 (See https://builds.apache.org/job/tapestry-trunk-freestyle/509/ ) TAP5-1637 : Use SoftReferences to cache Page instances, rather than an explicit janitor job hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1166889 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/SymbolConstants.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/PageSourceImpl.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/services/TapestryModule.java /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/services/AppModule.java
          Massimo Lusetti made changes -
          Link This issue duplicates TAP5-1616 [ TAP5-1616 ]
          Massimo Lusetti made changes -
          Link This issue duplicates TAP5-1616 [ TAP5-1616 ]
          Massimo Lusetti made changes -
          Link This issue is duplicated by TAP5-1616 [ TAP5-1616 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          1h 47m 1 Howard M. Lewis Ship 08/Sep/11 21:34

            People

            • Assignee:
              Howard M. Lewis Ship
              Reporter:
              Howard M. Lewis Ship
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development