VratislavHais commented on pull request #3074: URL: https://github.com/apache/camel-quarkus/pull/3074#issuecomment-919798649
I spoke with Alexandre and we came to an agreement to keep the problematic route creation as it is (create route and immediately list it). Also I have made the changes mentioned in the comments. One test is disabled until new version of camel get released (issue for it is here: https://github.com/apache/camel-quarkus/issues/3097 ). Unfortunately this PR does not check all the tasks in the issue it addresses. There are still missing tests for: - Batch consumer - KMS for encryption - JMS like selectors via filter + defaultVisibilityTimeout - Use/test with an AWS client from quarkus (can be done after deleteQueue get's enabled and tested -> required so that we do not leave a mess after the test execution) Next steps are obviously to cover the missing points and to enable the disabled test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
