Qpid
  1. Qpid
  2. QPID-4105

Performance test framework uses sessions non-thread-safely

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.17
    • Component/s: Java Performance Tests
    • Labels:
      None

      Description

      The performance test framework added in QPID-3936 uses JMS sessions from multiple threads if multiple participants are configured to use the same session.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        4m 13s 1 Philip Harvey 04/Jul/12 13:35
        In Progress In Progress Reviewable Reviewable
        5s 1 Philip Harvey 04/Jul/12 13:35
        Reviewable Reviewable Resolved Resolved
        2h 53m 1 Keith Wall 04/Jul/12 16:28
        Keith Wall made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Fix Version/s 0.17 [ 12320179 ]
        Fix Version/s Future [ 12315490 ]
        Resolution Fixed [ 1 ]
        Hide
        Keith Wall added a comment -

        Patch applied. No review comments.

        Show
        Keith Wall added a comment - Patch applied. No review comments.
        Philip Harvey made changes -
        Assignee Philip Harvey [ philharveyonline ] Keith Wall [ k-wall ]
        Hide
        Philip Harvey added a comment -

        please can you review the patch, and commit if you're happy.

        Show
        Philip Harvey added a comment - please can you review the patch, and commit if you're happy.
        Philip Harvey made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Philip Harvey made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Philip Harvey made changes -
        Field Original Value New Value
        Attachment 0002-QPID-4105-perf-test-tidy-up-code-changes.patch [ 12535081 ]
        Hide
        Philip Harvey added a comment -

        attached patch to fix the threading problem

        Show
        Philip Harvey added a comment - attached patch to fix the threading problem
        Philip Harvey created issue -

          People

          • Assignee:
            Keith Wall
            Reporter:
            Philip Harvey
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development