reta commented on a change in pull request #398: [Draft] SSE implementation 
using AsyncContext
URL: https://github.com/apache/cxf/pull/398#discussion_r180721693
 
 

 ##########
 File path: 
systests/rs-sse/rs-sse-base/src/main/java/org/apache/cxf/systest/jaxrs/sse/BookStore.java
 ##########
 @@ -124,6 +124,7 @@ public void stop() {
             broadcaster.broadcast(createStatsEvent(builder.name("book"), 
1000));
             broadcaster.broadcast(createStatsEvent(builder.name("book"), 
2000));
 
+            Thread.sleep(2000);
 
 Review comment:
   That's correct, but it shouldn't be an issue because the events are 
broadcasted through the different endpoints (SSE ones). The call to close the 
broadcaster will happen eventually, the test case waits some time for data to 
become available.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to