Tapestry 5
  1. Tapestry 5
  2. TAP5-1197

Eliminate page pooling using shared page instances that separate their structure from the mutable state

    Details

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

      Description

      This has been suggested before, but the recent changes to class transformation API makes it much more reasonable to accomplish.

      The goal here is to identify all transient or mutable state in the page and store it via the PerThreadManager. This will be invisible to user code; the pages will appear to be individual instances with internal state ... but in fact, it will be a single instance (per locale) with all internal, mutable state stored elsewhere.

      Because this changes the semantics of some aspects of the component class transformation pipeline, there will be a compatibility mode that will allow pages to be pooled as with 5.1, while any third party libraries that contribute workers update.

      Why do all this? For large applications with very complex pages, this will be a big win, as Tapestry has been shown to strain the limits of available JVM heap (surprising to me, but apparently true) once you have a few dozen (or hundred) page instances (of each page type) floating around.

        Activity

        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #153 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/153/)
        TAP5-1197: Remove the Defense class and convert existing code to use Java's assert keyword

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #153 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/153/ ) TAP5-1197 : Remove the Defense class and convert existing code to use Java's assert keyword
        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #154 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/154/)
        TAP5-1197: Only track the page's dirty count in when the page pool is enabled (it is meaningless in the non-pooling mode)

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #154 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/154/ ) TAP5-1197 : Only track the page's dirty count in when the page pool is enabled (it is meaningless in the non-pooling mode)
        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #155 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/155/)
        TAP5-1197: Use an internally generated unique Long key to access PerThreadValues (which will be a more efficient Map key than a String), and make some other small optimizations

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #155 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/155/ ) TAP5-1197 : Use an internally generated unique Long key to access PerThreadValues (which will be a more efficient Map key than a String), and make some other small optimizations
        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #157 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/157/)

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #157 (See http://hudson.zones.apache.org/hudson/job/tapestry-5.2-freestyle/157/ )

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development