ICEfaces
  1. ICEfaces
  2. ICE-956

Tree performance issue with selectInputDate in Tree nodes

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.5
          • Fix Version/s: 1.6.1
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            Operating System: Windows XP
            Platform: PC

            Description

            There are performance issues when using the selectInputDate component inside
            Tree Nodes.

            When the selectInputDate is used in large trees > 500 nodes an
            javax.servlet.ServletException: java.lang.OutOfMemoryError is caused.

            ------------------- dump --------------------

            javax.servlet.ServletException: java.lang.OutOfMemoryError
            com.icesoft.faces.webapp.xmlhttp.PersistentFacesServlet.service(PersistentFacesServlet.java:407)
            javax.servlet.http.HttpServlet.service(HttpServlet.java:802)

            root cause

            javax.faces.el.EvaluationException: java.lang.OutOfMemoryError
            com.sun.faces.el.ValueBindingImpl.setValue(ValueBindingImpl.java:248)
            com.sun.faces.lifecycle.RestoreViewPhase.doPerComponentActions(RestoreViewPhase.java:216)
            com.sun.faces.lifecycle.RestoreViewPhase.doPerComponentActions(RestoreViewPhase.java:209)
            com.sun.faces.lifecycle.RestoreViewPhase.doPerComponentActions(RestoreViewPhase.java:209)
            com.sun.faces.lifecycle.RestoreViewPhase.doPerComponentActions(RestoreViewPhase.java:209)
            com.sun.faces.lifecycle.RestoreViewPhase.doPerComponentActions(RestoreViewPhase.java:209)
            com.sun.faces.lifecycle.RestoreViewPhase.doPerComponentActions(RestoreViewPhase.java:209)
            com.sun.faces.lifecycle.RestoreViewPhase.execute(RestoreViewPhase.java:140)
            com.sun.faces.lifecycle.LifecycleImpl.phase(LifecycleImpl.java:268)
            com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:110)
            com.icesoft.faces.webapp.xmlhttp.PersistentFacesServlet.service(PersistentFacesServlet.java:382)
            javax.servlet.http.HttpServlet.service(HttpServlet.java:802)

            -------------------------------------------------------------------------

              Issue Links

              depends on

              Improvement - An improvement or enhancement to an existing feature or task. ICE-2038 selectInputDate renders invisible calendar when renderAsPopup, which decreases the performance when inside the dataTable or Tree

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Greg McCleary added a comment -

                Smaller trees < 50 nodes appear to perform in an acceptable time and do cause
                OutOfMemoryErrors.

                Show
                Greg McCleary added a comment - Smaller trees < 50 nodes appear to perform in an acceptable time and do cause OutOfMemoryErrors.
                Hide
                Permalink
                Greg McCleary added a comment -

                The performance issue with a selectInputDate "popup" can be worked around by
                using a non-popup selectInputDate and toggle the rendered flag using a button
                or commandLink that is located in the tree node "content" facet.

                Show
                Greg McCleary added a comment - The performance issue with a selectInputDate "popup" can be worked around by using a non-popup selectInputDate and toggle the rendered flag using a button or commandLink that is located in the tree node "content" facet.
                Hide
                Permalink
                Greg McCleary added a comment -

                (In reply to comment #1) should say do NOT cause OutOfMemoryErrors .
                > Smaller trees < 50 nodes appear to perform in an acceptable time and do NOT cause
                > OutOfMemoryErrors.
                >
                >

                Show
                Greg McCleary added a comment - (In reply to comment #1) should say do NOT cause OutOfMemoryErrors . > Smaller trees < 50 nodes appear to perform in an acceptable time and do NOT cause > OutOfMemoryErrors. > >
                Hide
                Permalink
                Adnan Durrani added a comment -

                ICE-2038 fixes this issue.

                Show
                Adnan Durrani added a comment - ICE-2038 fixes this issue.

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    Greg McCleary
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: