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/01/07 00:46:20 UTC
[8/8] incubator-kudu git commit: Fix faulty hybrid clock test case
Fix faulty hybrid clock test case
Change-Id: I42c270f67cfb33ac6649c0e266784ebf4fd7ccdc
Reviewed-on: http://gerrit.cloudera.org:8080/1719
Reviewed-by: David Ribeiro Alves <da...@cloudera.com>
Tested-by: Internal Jenkins
Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/2187df37
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/2187df37
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/2187df37
Branch: refs/heads/master
Commit: 2187df374c032e737c59fa70e32bdde58babfde5
Parents: abe9448
Author: Dan Burkert <da...@cloudera.com>
Authored: Wed Jan 6 11:47:45 2016 -0800
Committer: Dan Burkert <da...@cloudera.com>
Committed: Wed Jan 6 22:46:22 2016 +0000
----------------------------------------------------------------------
src/kudu/server/hybrid_clock-test.cc | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/2187df37/src/kudu/server/hybrid_clock-test.cc
----------------------------------------------------------------------
diff --git a/src/kudu/server/hybrid_clock-test.cc b/src/kudu/server/hybrid_clock-test.cc
index 9a408ba..d87ba46 100644
--- a/src/kudu/server/hybrid_clock-test.cc
+++ b/src/kudu/server/hybrid_clock-test.cc
@@ -75,9 +75,7 @@ TEST(MockHybridClockTest, TestMockedSystemClock) {
TEST_F(HybridClockTest, TestNow_ValuesIncreaseMonotonically) {
const Timestamp now1 = clock_->Now();
const Timestamp now2 = clock_->Now();
- ASSERT_GE(HybridClock::GetLogicalValue(now1), HybridClock::GetLogicalValue(now2));
- ASSERT_GE(HybridClock::GetPhysicalValueMicros(now1),
- HybridClock::GetPhysicalValueMicros(now1));
+ ASSERT_LT(now1.value(), now2.value());
}
// Tests the clock updates with the incoming value if it is higher.