Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.15
    • Component/s: Java Broker
    • Labels:
      None

      Description

      Implement a sorted queue which can sort messages using specified message property predefined on queue creation.

      The sorting performed is based on alphanumeric ascending string sort order and cannot be altered or overridden.

      The attribute defined as the "qpid.queue_sort_key" for a queue should be present in the custom properties of the message sent i.e. a value stored by a call to Message.setStringProperty(myProperty, "abc123") and retrieved with Message.getStringProperty(myProperty).

      The sort key value used must be of java type type String. JMS pre-defined properties can-not be used for sorting.

      Messages sent to a Sorted Queue without the specified attribute provided should be added to the end of the queue.

        Activity

        Hide
        Alex Rudyy added a comment -

        Attached patch with a sorted queue implementation.

        Show
        Alex Rudyy added a comment - Attached patch with a sorted queue implementation.
        Hide
        Alex Rudyy added a comment -

        Keith,
        Could you please review and commit the patch?

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

        Patch applied.

        Show
        Keith Wall added a comment - Patch applied.
        Hide
        Andrew MacBean added a comment -

        2 of the tests are not required or wanted, could you please apply patch to remove.

        Show
        Andrew MacBean added a comment - 2 of the tests are not required or wanted, could you please apply patch to remove.
        Hide
        Andrew MacBean added a comment -

        2 tests that are not wanted or required made it into the applied patch. have attached a correction patch.

        Show
        Andrew MacBean added a comment - 2 tests that are not wanted or required made it into the applied patch. have attached a correction patch.
        Hide
        Keith Wall added a comment -

        Applied second patch from Alex and Andrew.

        Show
        Keith Wall added a comment - Applied second patch from Alex and Andrew.

          People

          • Assignee:
            Keith Wall
            Reporter:
            Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development