You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2016/12/08 23:49:10 UTC

kudu git commit: Fix linked_list-test flakiness

Repository: kudu
Updated Branches:
  refs/heads/master e1acdfba6 -> f17562fd0


Fix linked_list-test flakiness

A recent patch changed this test to do a snapshot scan at the end
which is now failing with a timeout. That patch looped this test
a bunch but, apparently, jenkins is even more stressful than
--stress_cpu_threads=4. I re-looped the test with --stress_cpu_theads=8
and observed 8/1000 failures.

The fix is to increase the timeout. After I did that I looped the
tests again and 1000/1000 passed.

Config:
KUDU_ALLOW_SLOW_TESTS=1 ../../build-support/dist_test.py loop -n 1000 \
bin/linked_list-test --stress_cpu_threads=8 --gtest_filter=*TestLoadAndVerify*

Results:
http://dist-test.cloudera.org//job?job_id=david.alves.1481225032.13214

Change-Id: Ice3d92e8112fc336d24600d6d817e37729ff8467
Reviewed-on: http://gerrit.cloudera.org:8080/5424
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Reviewed-by: Todd Lipcon <to...@apache.org>


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

Branch: refs/heads/master
Commit: f17562fd0aa31dc61d6fe6ea877de5cd54f95c41
Parents: e1acdfb
Author: David Alves <dr...@apache.org>
Authored: Thu Dec 8 11:34:31 2016 -0800
Committer: Todd Lipcon <to...@apache.org>
Committed: Thu Dec 8 23:46:40 2016 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/linked_list-test-util.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/f17562fd/src/kudu/integration-tests/linked_list-test-util.h
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/linked_list-test-util.h b/src/kudu/integration-tests/linked_list-test-util.h
index 9e6ac01..876dc52 100644
--- a/src/kudu/integration-tests/linked_list-test-util.h
+++ b/src/kudu/integration-tests/linked_list-test-util.h
@@ -592,7 +592,7 @@ Status LinkedListTester::VerifyLinkedListRemote(
   client::KuduScanner scanner(table.get());
   RETURN_NOT_OK_PREPEND(scanner.SetProjectedColumns(verify_projection_), "Bad projection");
   RETURN_NOT_OK(scanner.SetBatchSizeBytes(0)); // Force at least one NextBatch RPC.
-  RETURN_NOT_OK(scanner.SetTimeoutMillis(20 * 1000 /* 20 seconds */));
+  RETURN_NOT_OK(scanner.SetTimeoutMillis(60 * 1000 /* 60 seconds */));
 
   if (snapshot_timestamp != kNoSnapshot) {
     RETURN_NOT_OK(scanner.SetReadMode(client::KuduScanner::READ_AT_SNAPSHOT));