zentol commented on pull request #17521: URL: https://github.com/apache/flink/pull/17521#issuecomment-963038702
I'm sorry, but that implementation sounds really unsafe and will only work for the simplest of use-cases. Let's say you have 2 of your plugin jars that both bundle the same dependency with incompatible version (let's say guava), and said dependency is used by the user (e.g., because it is exposed in the API of the connector). It's now a coin toss as to whether the correct guava version will be loaded. Actually, if we take an application cluster, it is equivalent to just adding the connectors to `lib/`, and in terms of classloader isolation, is generally also equivalent to just bundling everything in the user-jar. These kind of problems are exactly why we don't want to expose this to users. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
