Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Performance related problem:

      Both Mojarra and MyFaces use iterator() for iteration over children. That is very slow if view is big and contains thousands of components.

      Specify:
      1) Specify : UIComponent.getChildren() is type of java.util.RandomAccess
      2) Specify new UIComponent.getFacets(): List (java.util.RandomAccess) for fast iteration over facets

      1) is very easy because mojarra and myfaces use ArrayList for children (so does trinidad)

      I think this is not implementation detail, because custom renderkits must know that fast iteration is possible

      See:
      https://issues.apache.org/jira/browse/MYFACES-3130
      http://www.mail-archive.com/dev@myfaces.apache.org/msg52979.html

        Activity

        Hide
        Manfred Riem added a comment -

        Setting priority to Minor

        Show
        Manfred Riem added a comment - Setting priority to Minor
        Hide
        Ed Burns added a comment -

        Set priority to baseline ahead of JSF 2.3 triage. Priorities will be assigned accurately after this exercise.

        Show
        Ed Burns added a comment - Set priority to baseline ahead of JSF 2.3 triage. Priorities will be assigned accurately after this exercise.
        Hide
        Jakob Korherr added a comment -

        As I followed the discussion on MYFACES-3130, I have to say that this is a valueable "change" (actually it is not really a change, just a hint).

        So +1 on including this info in the spec from my side!

        Show
        Jakob Korherr added a comment - As I followed the discussion on MYFACES-3130, I have to say that this is a valueable "change" (actually it is not really a change, just a hint). So +1 on including this info in the spec from my side!

          People

          • Assignee:
            Unassigned
            Reporter:
            Martin Kočí
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: