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

Sijie Guo commented on BOOKKEEPER-709:
--------------------------------------

{code}
Thread.sleep(3000); // sleep a second to allow time to complete
{code}

if you change 1000 to 3000, please change corresponding comment.

BTW, I don't think it is a good idea to change method signature to throw 
UnknownHostException just for a debug message. otherwise, the callers have to 
catch an exception just for a debug statement, which makes the code a bit messy.

> SlowBookieTest#testSlowBookie fails intermittently
> --------------------------------------------------
>
>                 Key: BOOKKEEPER-709
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-709
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>             Fix For: 4.3.0
>
>         Attachments: 0001-BOOKKEEPER-709.patch
>
>
> SlowBookieTest#testSlowBookie fails intermittently when verifying the result 
> of addEntry. 
> {code}
> junit.framework.AssertionFailedError: expected:<0> but was:<-559038737>
>       at junit.framework.Assert.fail(Assert.java:47)
>       at junit.framework.Assert.failNotEquals(Assert.java:283)
>       at junit.framework.Assert.assertEquals(Assert.java:64)
>       at junit.framework.Assert.assertEquals(Assert.java:195)
>       at junit.framework.Assert.assertEquals(Assert.java:201)
>       at 
> org.apache.bookkeeper.client.SlowBookieTest.testSlowBookie(SlowBookieTest.java:82)
>       at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> {code}



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to