reswqa commented on code in PR #20229:
URL: https://github.com/apache/flink/pull/20229#discussion_r917869284
##########
flink-tests/src/test/java/org/apache/flink/test/accumulators/AccumulatorLiveITCase.java:
##########
@@ -142,18 +142,16 @@ public void testStreaming() throws Exception {
submitJobAndVerifyResults(jobGraph);
}
- private static void submitJobAndVerifyResults(JobGraph jobGraph) throws
Exception {
+ private void submitJobAndVerifyResults(JobGraph jobGraph) throws Exception
{
Review Comment:
Here is for use `getClass().getClassLoader()` in lambda expression, perhaps
it is more appropriate to use `AccumulatorLiveITCase.class.getClassLoader()` or
`CheckedThread.class.getClassLoader()` directly
--
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
