This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
The following commit(s) were added to refs/heads/master by this push:
new 4116941c0 KUDU-3591 Fix the flaky test
FsManagerTestBase.TestAddRemoveDataDirsFuzz
4116941c0 is described below
commit 4116941c082be5d6e3addfe3f6b6d2ea468416e4
Author: Yingchun Lai <laiyingc...@apache.org>
AuthorDate: Sat Jul 20 23:09:24 2024 +0800
KUDU-3591 Fix the flaky test FsManagerTestBase.TestAddRemoveDataDirsFuzz
Reduce the failure rate of the test by lowering its loop
times. For example, when FLAGS_block_manager == "logr",
opens a data directory will open a RocksDB instance, it
consumes more time than that if FLAGS_block_manager == "log".
Change-Id: I56a03d64c8ababd63fe31fa1e7fce1efac3875e8
Reviewed-on: http://gerrit.cloudera.org:8080/21604
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <ale...@apache.org>
---
src/kudu/fs/fs_manager-test.cc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/kudu/fs/fs_manager-test.cc b/src/kudu/fs/fs_manager-test.cc
index b030d1e03..75e26d917 100644
--- a/src/kudu/fs/fs_manager-test.cc
+++ b/src/kudu/fs/fs_manager-test.cc
@@ -1255,12 +1255,12 @@ TEST_P(FsManagerTestBase, TestAddRemoveDataDirsFuzz) {
#if defined(THREAD_SANITIZER) || defined(ADDRESS_SANITIZER)
// When using a sanitizer, reduce the loop times to get a more stable result.
- const int kNumAttempts = 50;
+ const int kNumAttempts = 10;
#else
// In some situations, the tests would last too long time, so we reduce the
loop times if not
// AllowSlowTests(). For example, when FLAGS_block_manager == "logr", opens
a data directory will
// open a RocksDB instance, it consumes more time than that if
FLAGS_block_manager == "log".
- const int kNumAttempts = AllowSlowTests() ? 1000 : 50;
+ const int kNumAttempts = AllowSlowTests() ? 1000 : 10;
#endif
Random rng_(SeedRandom());
