zentol opened a new pull request #17184: URL: https://github.com/apache/flink/pull/17184
At various points in Flink we use the ServiceLoader mechanism to load implementations, for example filesystems or reporters. It is thus only natural that we also have some test service implementations which are used in specific tests. However, these test implementations are intended to only be used in very specific tests, but are currently but on the classpath for all tests in that module (+all that depend on the test-jar of such a module). This causes confusion (e.g., suddenly there are 5 reporter factories available) or logging noise (e.g., custom netty handlers being loaded by each MiniCluster). This PR mplements a junit extension that runs the test with a customized classloader, which also has access to a temporary directory containing generated service entries. The RestExternalHandlersITCase was migrated to use this extension, as this was the issue that prompted the whole effort. -- 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
