tillrohrmann commented on issue #6996: [FLINK-10715] Change reporter log level URL: https://github.com/apache/flink/pull/6996#issuecomment-435330790 I think the PR fixes a valid problem which is the logging of a non-critical issue in a way that it seems to affect the proper functioning of Flink. This can be very confusing to our users and, thus, I think we should change it. Even after fixing the underlying problem of the reporters I think we should not report a Reporter failure as a warning since Flink can easily recover from that. The problem with the `ConcurrentModificationException` in the e2e tests is that they occur in tests where no exceptions should be thrown. Thus, disabling the log verification and adding some other test assertions to ensure the correctness of the test seems not right to me.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
