Skip to content

Conversation

@jeanouii
Copy link
Contributor

@jeanouii jeanouii commented Jan 22, 2026

Very simple test to valid expiration is serialized with HTTP protocol.
The test simply sends a message with TTL 100ms and waits 30s for the message to show up in the DLQ.
Unfortunately the broker side check happens every 30s, so on fast nodes, the receive returns before the broker actually sends the message to the DLQ.
Adding a policy to speed up the move to the DLQ. The test should also run way faster.

Copy link
Contributor

@cshannon cshannon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, as a side note we really need to work on our configuration as it's such a pain to do simple stuff like this.

I've had discussions with @jbonofre and @mattrpav already about some options (we are planning to remove Spring or make it optional at least in AMQ 7) and for the embedded side of configuration like in our unit tests, using a builder pattern makes the most sense and I think would greatly improve things so hopefully we can work towards that soon.

@jbonofre
Copy link
Member

Yes, that's clearly planned for ActiveMQ 7. We should use a configuration aggregator that clearly list and document all configurations.
Also the default configuration should be probably reviewed (again for AMQ 7 as it's potentially breaking change for users).

@jeanouii
Copy link
Contributor Author

I agree that Spring does not bring much to us. But we have to deal with dependency updates, cve, etc.

Having a broker that we can build with a builder pattern, like it's been done on other projects I've been involved in, would be neat and helpful.

@jbonofre
Copy link
Member

@jeanouii that's the PoC I started 😄

@jeanouii
Copy link
Contributor Author

Loolks like it's green!

@jbonofre
Copy link
Member

That's super nice !

@jbonofre jbonofre merged commit a991438 into apache:main Jan 22, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants