StephanEwen opened a new pull request #11120: [FLINK-16135][core] Unify the TemporaryClassLoaderContext and AutoContextClassLoader URL: https://github.com/apache/flink/pull/11120 ## What is the purpose of the change These two classes do basically same thing, this PR unifies the use to a consolidated implementation. The consolidated implementation - Resides in `org.apache.flink.util` (not in the plugins sub-package) because it is intended as a general util - Is called `TemporaryClassLoaderContext` (better name) - Uses `TemporaryClassLoaderContext.of(classLoader)` pattern from the `AutoContextClassLoader`. This pattern is more in line with the "don't do work in the constructor" coding guide. This also makes it more future-proof, in case we need to do extended handling in the future for specific classloaders (child-first, plugins) or more elaborate handling of thread local variables, etc.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
