You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by vv...@apache.org on 2020/05/29 13:50:49 UTC

[kafka] branch 2.6 updated: MINOR: Relax Percentiles test (#8748)

This is an automated email from the ASF dual-hosted git repository.

vvcephei pushed a commit to branch 2.6
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/2.6 by this push:
     new 9a16379  MINOR: Relax Percentiles test (#8748)
9a16379 is described below

commit 9a163794b6772443e6df265e2223929399ab3a6f
Author: A. Sophie Blee-Goldman <so...@confluent.io>
AuthorDate: Thu May 28 16:25:50 2020 -0700

    MINOR: Relax Percentiles test (#8748)
    
    Decrease test sensitivity to +/- 20% to hopefully eliminate flakiness.
    
    Reviewers: John Roesler <vv...@apache.org>
---
 .../src/test/java/org/apache/kafka/common/metrics/MetricsTest.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/clients/src/test/java/org/apache/kafka/common/metrics/MetricsTest.java b/clients/src/test/java/org/apache/kafka/common/metrics/MetricsTest.java
index e57d4e5..082513d 100644
--- a/clients/src/test/java/org/apache/kafka/common/metrics/MetricsTest.java
+++ b/clients/src/test/java/org/apache/kafka/common/metrics/MetricsTest.java
@@ -568,8 +568,8 @@ public class MetricsTest {
             double expectedP90 = values.get(p90Index - 1);
             double expectedP99 = values.get(p99Index - 1);
 
-            assertEquals(expectedP90, (Double) p90.metricValue(), expectedP90 / 10);
-            assertEquals(expectedP99, (Double) p99.metricValue(), expectedP99 / 10);
+            assertEquals(expectedP90, (Double) p90.metricValue(), expectedP90 / 5);
+            assertEquals(expectedP99, (Double) p99.metricValue(), expectedP99 / 5);
         } catch (AssertionError e) {
             throw new AssertionError("Assertion failed in randomized test. Reproduce with seed = " + seed + " .", e);
         }