Uploaded image for project: 'JBoss Transaction Manager'
  1. JBoss Transaction Manager
  2. JBTM-4022

Performance improvements identified while reviewing JBTM-4014

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Unresolved
    • Icon: Major Major
    • None
    • 7.3.3.Final
    • STM, Transaction Core

      Three areas were identified during the PR review of the Uid performance improvements added by JBTM-4014:

      1. The pid and inetAddr fields of Uid.java are calculated per instance even though they are effectively static (https://github.com/jbosstm/narayana/pull/2427#issuecomment-3485215583)
      2. ShadowingStore.remove_state makes 3 sys calls when just one can suffice (Files::deleteIfExist): seebcomment: https://github.com/jbosstm/performance/pull/185#issuecomment-3466164158
      3. Reliance on finalizers in the STM benchmark is reducing throughput: see the flame graph in PR comment https://github.com/jbosstm/performance/pull/185#issuecomment-3466164158

       

      When analysing the best way to take advantage of these opportunities I'd recommend opening a sub-task for each point addressed and assign ownership of the sub-task as appropriate.

              Unassigned Unassigned
              rhn-engineering-mmusgrov Michael Musgrove
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: