You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by an...@apache.org on 2018/07/18 13:49:27 UTC

zookeeper git commit: ZOOKEEPER-3074: Flaky test:org.apache.zookeeper.server.ServerStatsTest.testLatencyMetrics

Repository: zookeeper
Updated Branches:
  refs/heads/master cea251a18 -> 81b515549


ZOOKEEPER-3074: Flaky test:org.apache.zookeeper.server.ServerStatsTest.testLatencyMetrics

more details in [ZOOKEEPER-3074](https://issues.apache.org/jira/browse/ZOOKEEPER-3074)

Author: maoling <ma...@sina.com>

Reviewers: hanm@apache.org, andor@apache.org, nkalmar@cloudera.com

Closes #556 from maoling/ZOOKEEPER-3074


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

Branch: refs/heads/master
Commit: 81b51554922d4cee5dde4a848a106f286f4c407b
Parents: cea251a
Author: maoling <ma...@sina.com>
Authored: Wed Jul 18 15:49:21 2018 +0200
Committer: Andor Molnar <an...@apache.org>
Committed: Wed Jul 18 15:49:21 2018 +0200

----------------------------------------------------------------------
 .../test/org/apache/zookeeper/server/ServerStatsTest.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zookeeper/blob/81b51554/src/java/test/org/apache/zookeeper/server/ServerStatsTest.java
----------------------------------------------------------------------
diff --git a/src/java/test/org/apache/zookeeper/server/ServerStatsTest.java b/src/java/test/org/apache/zookeeper/server/ServerStatsTest.java
index 3757b76..aa207d1 100644
--- a/src/java/test/org/apache/zookeeper/server/ServerStatsTest.java
+++ b/src/java/test/org/apache/zookeeper/server/ServerStatsTest.java
@@ -26,7 +26,7 @@ import org.junit.Test;
 
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.mockito.Mockito.mock;
-import static org.hamcrest.Matchers.greaterThanOrEqualTo;
+import static org.hamcrest.Matchers.lessThanOrEqualTo;
 
 public class ServerStatsTest extends ZKTestCase {
 
@@ -73,11 +73,11 @@ public class ServerStatsTest extends ZKTestCase {
 
         // Then ...
         assertThat("Max latency check", 2000L,
-                greaterThanOrEqualTo(serverStats.getMaxLatency()));
+                lessThanOrEqualTo(serverStats.getMaxLatency()));
         assertThat("Min latency check", 1000L,
-                greaterThanOrEqualTo(serverStats.getMinLatency()));
+                lessThanOrEqualTo(serverStats.getMinLatency()));
         assertThat("Avg latency check", 1500L,
-                greaterThanOrEqualTo(serverStats.getAvgLatency()));
+                lessThanOrEqualTo(serverStats.getAvgLatency()));
 
         // When reset...
         serverStats.resetLatency();