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