Qpid
  1. Qpid
  2. QPID-4314

[Java Distributed Tests] Add performance tests measuring the impact of selectors on consumer performance

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.19
    • Fix Version/s: 0.19
    • Component/s: Java Performance Tests
    • Labels:
      None

      Description

      Add performance tests measuring the impact of selectors on consumer performance.

      Both overlapping and non-overlapping selectors need testing

        Activity

        Hide
        Robbie Gemmell added a comment -

        Resolving.

        Show
        Robbie Gemmell added a comment - Resolving.
        Hide
        Alex Rudyy added a comment - - edited

        Robbie,

        Could you please review the commit?

        Show
        Alex Rudyy added a comment - - edited Robbie, Could you please review the commit?
        Hide
        Robbie Gemmell added a comment -

        Hi Alex,

        Keith and I looked over the patch and noticed a couple of things it would be good to fix before commit.

        The consumers are set to evaluate latency, which we probably dont want for this test. Also, wouldnt it be possible to express this test more concisely using the JavaScript configuration? The file is pretty large and it appears that the selector values are the main thing that changes, which we could hopefully jsut express as a list and then iterate the values as the consumers are generated.

        Show
        Robbie Gemmell added a comment - Hi Alex, Keith and I looked over the patch and noticed a couple of things it would be good to fix before commit. The consumers are set to evaluate latency, which we probably dont want for this test. Also, wouldnt it be possible to express this test more concisely using the JavaScript configuration? The file is pretty large and it appears that the selector values are the main thing that changes, which we could hopefully jsut express as a list and then iterate the values as the consumers are generated.
        Hide
        Alex Rudyy added a comment -

        Robbie,
        Could you please review and commit the patch?

        Show
        Alex Rudyy added a comment - Robbie, Could you please review and commit the patch?
        Hide
        Alex Rudyy added a comment -

        Attached a patch adding new performance tests for consumers with selectors

        Show
        Alex Rudyy added a comment - Attached a patch adding new performance tests for consumers with selectors

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development