ActiveMQ
  1. ActiveMQ
  2. AMQ-4271

<virtualSelectorCacheBrokerPlugin/> should be able to cache consumers with no selectors as well

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.7.0
    • Fix Version/s: 5.9.0
    • Component/s: None
    • Labels:
      None

      Description

      With selectorAware=true setting for virtual topics we are checking for consumers (and their selectors) to detect if the message should be sent to the virtual topic or not. virtualSelectorCacheBrokerPlugin helps in the case we want to prevent "message loss" for subscribers that goes offline. But it doesn't work for the consumers that don't have selectors as they are not being cached. This should be sensible thing to do and allow both consumers with and without selectors to behave the same in this scenario.

        Issue Links

          Activity

          Hide
          Dejan Bosanac added a comment -

          Fixed with svn revision 1438032

          Show
          Dejan Bosanac added a comment - Fixed with svn revision 1438032
          Hide
          Adam Edwards added a comment -

          thanks! I have tested this against 5.8-SNAPSHOT and works well. Appreciate the enhancement.

          Show
          Adam Edwards added a comment - thanks! I have tested this against 5.8-SNAPSHOT and works well. Appreciate the enhancement.
          Hide
          Timothy Bish added a comment -

          @Dejan, it appears that you fixed this, is there more to do or can we resolve this one.

          Show
          Timothy Bish added a comment - @Dejan, it appears that you fixed this, is there more to do or can we resolve this one.
          Hide
          Dejan Bosanac added a comment -

          Resolved it. Thanks for the update, I must have forgotten to close it.

          Show
          Dejan Bosanac added a comment - Resolved it. Thanks for the update, I must have forgotten to close it.

            People

            • Assignee:
              Dejan Bosanac
              Reporter:
              Dejan Bosanac
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development