Github user rdblue commented on a diff in the pull request:

    https://github.com/apache/incubator-toree/pull/104#discussion_r121755733
  
    --- Diff: 
kernel/src/test/scala/org/apache/toree/kernel/protocol/v5/stream/KernelInputStreamSpec.scala
 ---
    @@ -65,6 +65,9 @@ class KernelInputStreamSpec
         // set of data
         
doReturn(system.actorSelection(fakeInputOutputHandlerActor.path.toString))
           .when(mockActorLoader).load(MessageType.Incoming.InputReply)
    +    // Allow time for the actors to start. This avoids read() hanging 
forever
    +    // when running tests in gradle.
    +    Thread.sleep(100)
    --- End diff --
    
    I do, too. The problem is that this test is flaky on my machine without it. 
We need to take a closer look at the flaky test, but I don't think this PR is 
the place to fix it. How about I open an issue for it and we solve the problem 
separately?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to