[GitHub] [flink] zentol edited a comment on pull request #17521: [FLINK-24558][API/DataStream]make parent ClassLoader variable which c…

[GitBox <gi...@apache.org>]

zentol edited a comment on pull request #17521:
URL: https://github.com/apache/flink/pull/17521#issuecomment-952669317


   I don't think it is a good idea to give users direct access to this part of the code. It just yet again increases the API surface, and for some _very_ important internal thing that we need to be able to change at a whim.
   
   Furthermore, I don't see yet how this would solve the issue at hand. The proposed interface provides no differentiating factor that could be used to create different classloaders for each task (like the Task ID). Even then, the classloader is shared across different tasks running on the same TM, so it would have to behave the same anway? Given that they all have access to the same jars, I'm curious how the behavior is supposed to be different in the first place.
   
   All in all, I think this needs way more discussion.


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org