kulagaIA commented on PR #12669: URL: https://github.com/apache/camel/pull/12669#issuecomment-1878563843
> In general, the `StopWatch` (or the standard `Duration` from Java) should be used for computing duration. > > Looking at the comments on this test ... I think the right approach is to disable it on github and leave the logic as is. > > So, my .2 cents: just disable the test on Github (search for `@DisabledIfSystemProperty` on the code base to see how we do it). It seems that using StopWatch and measuring elapsed time after .assertIsSatisfied(context) was called, instead of calcualting elapsed time using exchange clock, helped to make this test at least less flacky. I think it can be merged, and if I see it failing again, I'll disable. -- 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
