Qpid
  1. Qpid
  2. QPID-3518

Introduce client side ability to detect server side selector support presence

    Details

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

      Description

      Currently the Java Client supports client-side selectors and the Java Broker provides server-side selectors. Where available, server-side selectors should always be used. On the C++ side, there are only client-side selectors and server-side selectors remain a future enhancement.

      The Java Broker needs to advertise, on connection, the presence of server-side selectors and the Java Client detect this and use them. Where the Java Client does not see this feature advertised (i.e. on connection to the C++ broker) it should revert to use client-side selectors.

      This detection approach should be implemented in a generic way such that the client detects on connection a list of advertised features from the broker, so that it could be extended to support XA and other such features.

      The model adopted should be consistent across Java and C++ brokers.

        Activity

        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.
        Alex Rudyy made changes -
        Fix Version/s 0.15 [ 12319043 ]
        Keith Wall made changes -
        Fix Version/s 0.13 [ 12316854 ]
        Alex Rudyy made changes -
        Assignee Alex Rudyy [ alex.rufous ] Keith Wall [ k-wall ]
        Hide
        Alex Rudyy added a comment -

        Keith, could you please review and commit attached patch?

        Show
        Alex Rudyy added a comment - Keith, could you please review and commit attached patch?
        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 -
        Hide
        Alex Rudyy added a comment -

        Attached a patch adding the client side ability to detect server side selector support

        Show
        Alex Rudyy added a comment - Attached a patch adding the client side ability to detect server side selector support
        Alex Rudyy made changes -
        Field Original Value New Value
        Assignee Alex Rudyy [ alex.rufous ]
        Robbie Gemmell created issue -

          People

          • Assignee:
            Keith Wall
            Reporter:
            Robbie Gemmell
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development