XMLWordPrintable

    Details

    • Type: Enhancement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: EAP_EWP 5.2.0
    • Fix Version/s: EAP_EWP 5.3.0.ER2
    • Component/s: JCA
    • Labels:
      None
    • Affects:
      Release Notes
    • Release Notes Text:
      Hide
      <para>A method-level synchronized block was added as part of the fix for JBPAPP-7464, however this created a concurrency bottleneck which limited connection pools to creating one connection at a time. This issue was fixed by removing the method synchronization and making it thread safe in an alternate way. As a result, more than one connection can now be created concurrently.</para>
      Show
      <para>A method-level synchronized block was added as part of the fix for JBPAPP-7464 , however this created a concurrency bottleneck which limited connection pools to creating one connection at a time. This issue was fixed by removing the method synchronization and making it thread safe in an alternate way. As a result, more than one connection can now be created concurrently.</para>
    • Release Notes Docs Status:
      Documented as Resolved Issue
    • Docs QE Status:
      NEW

      Description

      JBPAPP-7464 was fixed by using synchronized to control access to the properties, but the synchronization is wider than it needs to be. The entire ManagedConnectionFactory.createManagedConnection() method is synchronized, when in reality only the part touching the properties needs to be. This causes a bottleneck where only one connection can be created at once, which impacts early use when the pool is filling.

      Backporting the relevant changes which are part of https://github.com/ironjacamar/ironjacamar/commit/e2db9c25fff32207c733c0945e825cd126f34ffb would remove the bottleneck.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  jameslivingston James Livingston
                  Reporter:
                  jameslivingston James Livingston
                  Writer:
                  Lucas Costi
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: