This is an automated email from the ASF dual-hosted git repository. michaelsmith pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit c01986d9e826c294773fdd7f9f34cb2f7c1c2b1f Author: jasonmfehr <[email protected]> AuthorDate: Wed Mar 27 19:30:19 2024 -0700 IMPALA-12945: Fix Flaky Ticker Test The ticker test failed on an exhaustive release build. The failure was because the total elapsed time was longer than expected. This change bumps up the margin of error from 1% to 2% to give slightly more time on the more intensive builds. Change-Id: Ibc8cf03ae68fb3103c5bbc438c32f6565b8c406c Reviewed-on: http://gerrit.cloudera.org:8080/21214 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- be/src/util/ticker-test.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/be/src/util/ticker-test.cc b/be/src/util/ticker-test.cc index 08fe8d6a3..469a19dc3 100644 --- a/be/src/util/ticker-test.cc +++ b/be/src/util/ticker-test.cc @@ -91,9 +91,9 @@ TEST(TickerTest, GenericTickerHappyPath) { fixture.Join(); EXPECT_EQ(cntr, 10); - // Include a 5 millisecond (1%) margin of error to tolerate differences in the + // Include a 10 millisecond (2%) margin of error to tolerate differences in the // precision of time measurements. - EXPECT_NEAR(NsToMs(sw.ElapsedTime()), static_cast<float>(50), 5); + EXPECT_NEAR(NsToMs(sw.ElapsedTime()), static_cast<float>(50), 10); } // Tests the case where the wakeup guard is not reset by the consuming code.
