kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t...@apache.org
Subject [1/3] kudu git commit: delete_table_test: improve test runtime in ASAN/TSAN
Date Thu, 22 Dec 2016 15:00:35 GMT
Repository: kudu
Updated Branches:
  refs/heads/master b9dadad6b -> 1179bbdb4


delete_table_test: improve test runtime in ASAN/TSAN

TestDeleteTableWhileScanInProgress was running almost 10 minutes in ASAN
builds. This was causing the suite to time out and show up on the flaky
list.

Change-Id: I255fedd9ddd13f0e3351af6e7dcd6bdab4bc945d
Reviewed-on: http://gerrit.cloudera.org:8080/5544
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <aserbin@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/ca2be41e
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/ca2be41e
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/ca2be41e

Branch: refs/heads/master
Commit: ca2be41ef91bd5545f91090fb390a9e779e29e8e
Parents: b9dadad
Author: Todd Lipcon <todd@apache.org>
Authored: Mon Dec 19 17:46:35 2016 +0700
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Mon Dec 19 19:02:43 2016 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/delete_table-test.cc | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/ca2be41e/src/kudu/integration-tests/delete_table-test.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/delete_table-test.cc b/src/kudu/integration-tests/delete_table-test.cc
index a2f6091..2c47a42 100644
--- a/src/kudu/integration-tests/delete_table-test.cc
+++ b/src/kudu/integration-tests/delete_table-test.cc
@@ -1346,7 +1346,12 @@ TEST_F(DeleteTableTest, TestDeleteTableWhileScanInProgress) {
 
   // Approximate number of rows to insert. This is not exact number due to the
   // way how the test controls the progress of the test workload.
+#if defined(ADDRESS_SANITIZER) || defined(THREAD_SANITIZER)
+  // Test is too slow in ASAN/TSAN.
+  const size_t rows_to_insert = 10000;
+#else
   const size_t rows_to_insert = AllowSlowTests() ? 100000 : 10000;
+#endif
   for (const auto sel : replica_selectors) {
     for (const auto mode : read_modes) {
       SCOPED_TRACE(Substitute("mode $0; replica $1",


Mime
View raw message