CAMEL-18376: camel-pulsar: add redelivery backoff for ack timeout, nack
Resolves the following Jira issue: https://issues.apache.org/jira/browse/CAMEL-18376
This PR adds in the ability to add a RedeliveryBackoff for both the ack timeout and negative acknowledgement cases (details linked). Modifications to existing tests and the addition of a new test should ensure that the backoff works as expected.
Checklist:
- [x] Make sure there is a JIRA issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes.
- [x] Each commit in the pull request should have a meaningful subject line and body.
- [x] If you're unsure, you can format the pull request title like
[CAMEL-XXX] Fixes bug in camel-file component, where you replaceCAMEL-XXXwith the appropriate JIRA issue. - [x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
- [ ] Run
mvn clean install -Psourcecheckin your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically. Below are the contribution guidelines: https://github.com/apache/camel/blob/main/CONTRIBUTING.md
:warning: This PR changes Camel components and will be tested automatically.
:x: Finished component verification: 1 component(s) test failed out of 1 component(s) tested
Will take a peek at job failures and fix shortly.
:x: Finished component verification: 1 component(s) test failed out of 5 component(s) tested
If there's another test failure, do you mind attaching the logs (or showing me where I can find them), @orpiske? My local machine seems to have trouble running these Docker-based tests.
:x: Finished component verification: 1 component(s) test failed out of 5 component(s) tested
If there's another test failure, do you mind attaching the logs (or showing me where I can find them), @orpiske? My local machine seems to have trouble running these Docker-based tests.
Access to the test logs is restricted by the organization, but I can upload them here as needed. test-logs(1).zip
:x: Finished component verification: 1 component(s) test failed out of 5 component(s) tested
:x: Finished component verification: 1 component(s) test failed out of 5 component(s) tested
@orpiske mind if I get logs for the last failed run?
:x: Finished component verification: 1 component(s) test failed out of 5 component(s) tested
Thanks for sending the files; I managed to commandeer a coworker's laptop running on x86_64 to avoid the local issues I was seeing (since I'm on an M1 it revolves around https://github.com/apache/pulsar/issues/12944). I left configuring container startup/timeouts, etc. out of the loop for now. Tests pass locally so hopefully this should be the last commit!
:heavy_check_mark: Finished component verification: 0 component(s) test failed out of 5 component(s) tested
Yeah it works now, thanks for contributing to Camel. You are welcome to keep improving ;)