-
Notifications
You must be signed in to change notification settings - Fork 118
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.
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
Triggering timer less often and fixing few typos #183
Triggering timer less often and fixing few typos #183
Conversation
@spatnity Thanks for contribution. Wouldn't it be more robust to have the value of |
Can anyone give me some idea on why some checks are failing here? |
@spatnity Please see #182 (comment) and I think we need to wait @jamesnetherton to do upgrade |
Thanks @zhfeng , got it! |
@spatnity Meanwhile, it could be good opportunity to implement a timer Feel welcome to ask questions if reason to do this or implementation is not clear :) |
Yeah sure looking into it. |
@aldettinger may i know what does i mean when u say test and prod environment? |
@spatnity Yes sure. Let's try to explain better. In the mentioned commit, the period endpoint query option of the timer component has value However, during tests we also wait for 10s at least, it's quite long. So the idea would be to have
|
Note that the
main
branch points at the latest stable Camel Quarkus release.Pull requests should be generally send against the
camel-quarkus-main
branch pointing at the current Camel Quarkus SNAPSHOT.