XComp commented on code in PR #22987: URL: https://github.com/apache/flink/pull/22987#discussion_r1274576162
########## flink-runtime/src/test/java/org/apache/flink/runtime/rest/RestClientTest.java: ########## @@ -207,6 +210,40 @@ public void testRestClientClosedHandling() throws Exception { } } + /** + * Tests that the futures returned by {@link RestClient} fail immediately if the client is + * already closed. + * + * <p>See FLINK-32583 + */ + @Test + public void testCloseClientBeforeRequest() throws Exception { Review Comment: that's a fair point - I know what you mean. :-D You could use JavaDoc to underline that. But using the compiler as a safeguard is a better option, I guess. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org