kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t...@apache.org
Subject [8/8] incubator-kudu git commit: Fix faulty hybrid clock test case
Date Wed, 06 Jan 2016 23:46:20 GMT
Fix faulty hybrid clock test case

Change-Id: I42c270f67cfb33ac6649c0e266784ebf4fd7ccdc
Reviewed-on: http://gerrit.cloudera.org:8080/1719
Reviewed-by: David Ribeiro Alves <david.alves@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 <dan@cloudera.com>
Authored: Wed Jan 6 11:47:45 2016 -0800
Committer: Dan Burkert <dan@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.


Mime
View raw message