Uploaded image for project: 'ActiveMQ'
  1. ActiveMQ
  2. AMQ-4271

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

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: 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
          dejanb Dejan Bosanac added a comment -

          Fixed with svn revision 1438032

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

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

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

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

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

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

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development