[ts_recovery-itest] Reduce flakyness of TestCrashBeforeWriteLogSegmentHeader

This test expects a log roll before crashing, but with log compression
enabled we can actually write a lot of data before rolling. This is
making the test fail later with a timeout on Restart().

This patch disables log compression for this test, making sure that
we don't write as much and thus making restart faster.

Change-Id: I208fc1fb05dcb0cd6f5386792efc66e5e24272de
Reviewed-on: http://gerrit.cloudera.org:8080/6279
Reviewed-by: Todd Lipcon <[email protected]>
Tested-by: Kudu Jenkins


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

Branch: refs/heads/master
Commit: 6c2c7c3a2954e0ac9a84fa366fc12d914a44cf4f
Parents: f7fa057
Author: David Alves <[email protected]>
Authored: Mon Mar 6 17:32:42 2017 -0800
Committer: David Ribeiro Alves <[email protected]>
Committed: Tue Mar 7 02:06:36 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/ts_recovery-itest.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/6c2c7c3a/src/kudu/integration-tests/ts_recovery-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/ts_recovery-itest.cc 
b/src/kudu/integration-tests/ts_recovery-itest.cc
index 4c430c9..b8e1a78 100644
--- a/src/kudu/integration-tests/ts_recovery-itest.cc
+++ b/src/kudu/integration-tests/ts_recovery-itest.cc
@@ -199,7 +199,7 @@ TEST_F(TsRecoveryITest, TestCrashDuringLogReplay) {
 // but before writing its header, the TS would previously crash on restart.
 // Instead, it should ignore the uninitialized segment.
 TEST_F(TsRecoveryITest, TestCrashBeforeWriteLogSegmentHeader) {
-  NO_FATALS(StartCluster({ "--log_segment_size_mb=1" }));
+  NO_FATALS(StartCluster({ "--log_segment_size_mb=1", 
"--log_compression_codec=NO_COMPRESSION" }));
 
   TestWorkload work(cluster_.get());
   work.set_num_replicas(1);

Reply via email to