MyFaces Core
  1. MyFaces Core
  2. MYFACES-3423

[perf] Use ArrayList instead LinkedList in UIComponent.pushComponentToEL / popComponentFromEL

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.12, 2.1.6
    • Component/s: JSR-314
    • Labels:
      None

      Description

      Doing some performance tests it was found that LinkedList create and discard lots of instances of LinkedList.EntrySet. Use an ArrayList is better because it does not create such instances, improving code speed and reducing memory footprint. Note use LinkedList was a good idea because that makes easier to think about algorithm itself, but since that part of the code is pretty stable and have , we can change it without problem.

        Activity

        Leonardo Uribe created issue -
        Leonardo Uribe made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.0.12 [ 12319175 ]
        Fix Version/s 2.1.6 [ 12319173 ]
        Resolution Fixed [ 1 ]
        Leonardo Uribe made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Leonardo Uribe
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development