You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/06/20 09:26:48 UTC

[GitHub] [flink] wuchong commented on pull request #20003: [FLINK-28080][runtime] Introduce MutableURLClassLoader as parent class of FlinkUserClassLoader and SafetyNetWrapperClassLoader

wuchong commented on PR #20003:
URL: https://github.com/apache/flink/pull/20003#issuecomment-1160201207

   Hi @zentol , we considered this option before. The problem is that the classloader is held by many components in table, e.g. `CatalogManger`, `FunctionCatalog`, `Planner`, and so on. It's hard to replace all the classloader references when adding a JAR URL. It's very easy to miss updating classloader in a certain component and cause problems. 
   
   That's why we prefer to introduce a `MutableURLClassLoader` which allows us to update the classloader in place. 


-- 
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