MB-21790: CheckpointTest.SeqnoAndHLCOrdering needs 1 checkpoint 50/70350/2
authorJim Walker <jim@couchbase.com>
Thu, 24 Nov 2016 16:45:38 +0000 (16:45 +0000)
committerDave Rigby <daver@couchbase.com>
Fri, 25 Nov 2016 07:56:47 +0000 (07:56 +0000)
Configure the test so that a single checkpoint is created, otherwise
the time-based closing code may split the data over more than 1
checkpoint causing the final data checks to fail.

Change-Id: I449a80d775b8d21afd9b46387a432dce7dcec2d1
Reviewed-on: http://review.couchbase.org/70350
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>
tests/module_tests/checkpoint_test.cc

index 38f1ca9..e8f529e 100644 (file)
@@ -884,9 +884,12 @@ TEST_F(CheckpointTest, SeqnoAndHLCOrdering) {
     const int n_items = 1000;
 
     // configure so we can store a large number of items
+    // configure with 1 checkpoint to ensure the time-based closing
+    // does not split the items over many checkpoints and muddy the final
+    // data checks.
     checkpoint_config = CheckpointConfig(DEFAULT_CHECKPOINT_PERIOD,
                                          n_threads*n_items,
-                                         /*numCheckpoints*/2,
+                                         /*numCheckpoints*/1,
                                          /*itemBased*/true,
                                          /*keepClosed*/false,
                                          /*enableMerge*/false);