You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by mp...@apache.org on 2017/04/19 01:02:28 UTC

[3/3] kudu git commit: log-test: Add additional assert

log-test: Add additional assert

This just adds an additional check which is nice to have in this test.
This was added in a backport to the 1.2.x branch and so is also being
added here.

Change-Id: Idb4847cb3fa8d4600be8a9b1b681613046988018
Reviewed-on: http://gerrit.cloudera.org:8080/6674
Tested-by: Mike Percy <mp...@apache.org>
Reviewed-by: Alexey Serbin <as...@cloudera.com>


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

Branch: refs/heads/master
Commit: e56f81b5d4e97deff3d071e1e3bfa421485d73ff
Parents: 6200da9
Author: Mike Percy <mp...@apache.org>
Authored: Tue Apr 18 11:47:46 2017 -0700
Committer: Mike Percy <mp...@apache.org>
Committed: Wed Apr 19 01:01:43 2017 +0000

----------------------------------------------------------------------
 src/kudu/consensus/log-test.cc | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/e56f81b5/src/kudu/consensus/log-test.cc
----------------------------------------------------------------------
diff --git a/src/kudu/consensus/log-test.cc b/src/kudu/consensus/log-test.cc
index 9098479..f37f60e 100644
--- a/src/kudu/consensus/log-test.cc
+++ b/src/kudu/consensus/log-test.cc
@@ -1017,6 +1017,7 @@ TEST_P(LogTestOptionalCompression, TestReadReplicatesHighIndex) {
   ASSERT_OK(reader->ReadReplicatesInRange(first_log_index, first_log_index + kSequenceLength - 1,
                                           LogReader::kNoSizeLimit, &replicates));
   ASSERT_EQ(kSequenceLength, replicates.size());
+  ASSERT_GT(op_id.index(), std::numeric_limits<int32_t>::max());
 }
 
 // Test various situations where we expect different segments depending on what the