[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-3470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16901411#comment-16901411
 ] 

Hudson commented on ZOOKEEPER-3470:
-----------------------------------

SUCCESS: Integrated in Jenkins build Zookeeper-trunk-single-thread #486 (See 
[https://builds.apache.org/job/Zookeeper-trunk-single-thread/486/])
ZOOKEEPER-3470: fix flaky LearnerMetricsTest (andor: rev 
9fa18ab0e1ce07b3cc61e8758e27d393253b81fb)
* (edit) 
zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Follower.java
* (edit) 
zookeeper-server/src/main/java/org/apache/zookeeper/server/FinalRequestProcessor.java
* (edit) zookeeper-server/src/test/java/org/apache/zookeeper/ZKTestCase.java
* (edit) 
zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/LearnerMetricsTest.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/Request.java


> Flaky test: LearnerMetricsTest.testLearnerMetricsTest()
> -------------------------------------------------------
>
>                 Key: ZOOKEEPER-3470
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3470
>             Project: ZooKeeper
>          Issue Type: Sub-task
>          Components: tests
>    Affects Versions: 3.6.0
>            Reporter: Andor Molnar
>            Assignee: Szalay-Beko Mate
>            Priority: Major
>              Labels: flaky, flaky-test, newbie, pull-request-available, test
>             Fix For: 3.6.0
>
>          Time Spent: 3h 10m
>  Remaining Estimate: 0h
>
> Hi team,
> New test testLearnerMetricsTest() added by the following commit failed 2 
> times on master with the same error:
> junit.framework.AssertionFailedError: expected:<10> but was:<9>
>  at 
> org.apache.zookeeper.server.quorum.LearnerMetricsTest.testLearnerMetricsTest(LearnerMetricsTest.java:88)
>  at 
> org.apache.zookeeper.JUnit4ZKTestRunner$LoggedInvokeMethod.evaluate(JUnit4ZKTestRunner.java:80)
> [https://github.com/apache/zookeeper/pull/856]
> Submitted by: jhuan31
> Maybe we should just blame the Thread.sleep(200) in line:77 and replace it 
> with some clever logic.
> Please take a look.
> Regards,
> Andor



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to