Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.15
    • Component/s: Broker-J
    • 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.rufous Alex Rudyy added a comment -

        Attached patch with a sorted queue implementation.

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

        Keith,
        Could you please review and commit the patch?

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

        Patch applied.

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

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

        Show
        macbean Andrew MacBean added a comment - 2 of the tests are not required or wanted, could you please apply patch to remove.
        Hide
        macbean 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
        macbean 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
        k-wall Keith Wall added a comment -

        Applied second patch from Alex and Andrew.

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

          People

          • Assignee:
            k-wall Keith Wall
            Reporter:
            alex.rufous Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development