Details

    • Type: New Feature New Feature
    • Status: Closed
    • 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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Reviewable Reviewable Open Open
        16m 16s 1 Alex Rudyy 17/Nov/11 18:25
        Open Open In Progress In Progress
        1d 8h 14m 2 Alex Rudyy 17/Nov/11 18:39
        In Progress In Progress Reviewable Reviewable
        20s 2 Alex Rudyy 17/Nov/11 18:39
        Reviewable Reviewable Resolved Resolved
        3d 4h 57m 1 Keith Wall 20/Nov/11 23:37
        Resolved Resolved Reopened Reopened
        10h 28m 1 Andrew MacBean 21/Nov/11 10:05
        Reopened Reopened Resolved Resolved
        49m 41s 1 Keith Wall 21/Nov/11 10:55
        Resolved Resolved Closed Closed
        1178d 9h 11m 1 Rob Godfrey 11/Feb/15 20:06
        Rob Godfrey made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Keith Wall made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Andrew MacBean made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        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.
        Andrew MacBean made changes -
        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.
        Keith Wall made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Keith Wall added a comment -

        Patch applied.

        Show
        Keith Wall added a comment - Patch applied.
        Andrew MacBean made changes -
        Andrew MacBean made changes -
        Attachment 0001-QPID-3622-Add-Sorted-Queue-funtionality.patch [ 12504191 ]
        Andrew MacBean made changes -
        Attachment 0001-QPID-3622-Add-Sorted-Queue-funtionality.patch [ 12504191 ]
        Alex Rudyy made changes -
        Attachment 0001-QPID-3622-Add-Sorted-Queue-funtionality.patch [ 12504091 ]
        Alex Rudyy made changes -
        Comment [ Wrong patch is attached ]
        Alex Rudyy made changes -
        Assignee Alex Rudyy [ alex.rufous ] Keith Wall [ k-wall ]
        Alex Rudyy made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Alex Rudyy made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Alex Rudyy made changes -
        Attachment 0001-QPID-3622-Add-Sorted-Queue-funtionality.patch [ 12504091 ]
        Alex Rudyy made changes -
        Attachment 0001-QPID-3622-Add-Sorted-Queue-funtionality.patch [ 12504085 ]
        Alex Rudyy made changes -
        Status Ready To Review [ 10006 ] Open [ 1 ]
        Assignee Keith Wall [ k-wall ] Alex Rudyy [ alex.rufous ]
        Alex Rudyy made changes -
        Assignee Alex Rudyy [ alex.rufous ] Keith Wall [ k-wall ]
        Alex Rudyy made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Alex Rudyy made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Alex Rudyy made changes -
        Assignee Keith Wall [ k-wall ] Alex Rudyy [ alex.rufous ]
        Alex Rudyy made changes -
        Assignee Keith Wall [ k-wall ]
        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?
        Alex Rudyy made changes -
        Field Original Value New Value
        Attachment 0001-QPID-3622-Add-Sorted-Queue-funtionality.patch [ 12504085 ]
        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.
        Alex Rudyy created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development