[kudu] 02/02: [subprocess] fix flakiness in TestEchoSubprocess

[alexey]

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

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit e01538b84ec14a49857c119d172e084a979775db
Author: Alexey Serbin <ale...@apache.org>
AuthorDate: Sat May 1 11:09:01 2021 -0700

    [subprocess] fix flakiness in TestEchoSubprocess
    
    This patch addresses flakiness in the testSlowWriterMetrics scenarios
    of the TestEchoSubprocess test.  I just increased the margin for the
    allowed timing discrepancy (BUFFER_MS), which is still under a
    reasonable threshold given amount of delay injected (BLOCK_MS).
    
    Change-Id: I82bb51b2e687b8ff63606ed0a0ebcb711df3ff41
    Reviewed-on: http://gerrit.cloudera.org:8080/17381
    Tested-by: Alexey Serbin <aser...@cloudera.com>
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
    Reviewed-by: Grant Henke <granthe...@apache.org>
---
 .../test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java
 
b/java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java
index 83c0eb9..92010c0 100644
--- 
a/java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java
+++ 
b/java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java
@@ -188,7 +188,7 @@ public class TestEchoSubprocess extends SubprocessTestUtil {
 
     // NOTE: timing on the exact slept time sometimes yields a small error, so
     // leave some buffer in checking for correctness.
-    final int BUFFER_MS = 5;
+    final int BUFFER_MS = 50;
     m = receiveResponse().getMetrics();
     Assert.assertEquals(1, m.getOutboundQueueLength());
     Assert.assertTrue(