Skip to content

ARTEMIS-5854 consumer mngmnt API not working #6196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
tabish121 merged 2 commits into from
Feb 2, 2026
Merged

ARTEMIS-5854 consumer mngmnt API not working #6196

tabish121 merged 2 commits into from
Feb 2, 2026

Conversation

@jbertram
Copy link
Contributor

@jbertram jbertram commented Jan 27, 2026

This commit does the following:

  • Allows lastDeliveredTime and lastAcknowledgedTime to be filtered via the management API.
  • Adds unit tests for all such attributes to verify they are filterable.
  • Looks up the correct value on the server when filtering on messagesDelivered.
  • Breaks up ViewTest into corresponding classes so all attributes can be unit tested eventually.

@jbertram jbertram force-pushed the ARTEMIS-5854 branch 2 times, most recently from 559213b to d055f97 Compare January 27, 2026 15:52
@tabish121
Copy link
Contributor

tabish121 commented Jan 30, 2026

I ran this in CI and the follow tests failed:

Run / Test / 4 / org.apache.activemq.artemis.tests.integration.management.ActiveMQServerControlTest.testListConsumersLastDeliveredTime() legacyCreateQueue=true
Run / Test / 4 / org.apache.activemq.artemis.tests.integration.management.ActiveMQServerControlTest.testListConsumersLastDeliveredTime() legacyCreateQueue=false
Run / Test / 0 / org.apache.activemq.artemis.tests.integration.management.ActiveMQServerControlUsingCoreTest.testListConsumersLastDeliveredTime() legacyCreateQueue=true
Run / Test / 0 / org.apache.activemq.artemis.tests.integration.management.ActiveMQServerControlUsingCoreTest.testListConsumersLastDeliveredTime() legacyCreateQueue=false
Run / Test / 0 / org.apache.activemq.artemis.tests.integration.management.ActiveMQServerControlUsingCoreTest.testListConsumersLastAckTime() legacyCreateQueue=true
Run / Test / 0 / org.apache.activemq.artemis.tests.integration.management.ActiveMQServerControlUsingCoreTest.testListConsumersLastAckTime() legacyCreateQueue=false

@jbertram
ARTEMIS-5854 consumer mngmnt API not working
f3991b4 
This commit does the following:

 - Allows `lastDeliveredTime` and `lastAcknowledgedTime` to be filtered
   via the management API.
 - Adds unit tests for all such attributes to verify they are
   filterable.
 - Looks up the correct value on the server when filtering on
   `messagesDelivered`.
 - Breaks up `ViewTest` into corresponding classes so all attributes can
   be unit tested eventually.
@jbertram jbertram force-pushed the ARTEMIS-5854 branch 5 times, most recently from 736393d to f79a972 Compare February 2, 2026 01:42
@jbertram
Copy link
Contributor Author

jbertram commented Feb 2, 2026

The test issues should be sorted now.

@tabish121 tabish121 merged commit d657f96 into apache:main Feb 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jbertram @tabish121 @andytaylor