adoroszlai opened a new pull request, #1012: URL: https://github.com/apache/ratis/pull/1012
## What changes were proposed in this pull request? Fix intermittent timeout in `TestStreamObserverWithTimeout#testWithTimeout`. 1. The first request may take long to reach the server, probably due to some initialization overhead. The patch adds a "warmup" request, for which timeout is not applied. 2. Client and server seem to share `grpc-default-executor` thread pool. This may result in delay at client due to server sleeping. The patch uses a thread per request in `GrpcTestServer`. 3. With these, we can restore test settings prior to 29d032dde65cfb441e7c912b1621cfff6c266674. https://issues.apache.org/jira/browse/RATIS-1989 ## How was this patch tested? Passed 500x in [repeated run](https://github.com/adoroszlai/ratis/actions/runs/7528357858) (although it [also passed](https://github.com/adoroszlai/ratis/actions/runs/7529250422) 500x on `master`). Also passed 500x locally. Regular CI: https://github.com/adoroszlai/ratis/actions/runs/7528350705 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
