ammu20-dev commented on PR #25656:
URL: https://github.com/apache/flink/pull/25656#issuecomment-2585004128

   > In that case the primary question should be - why does the 
`userClassLoader` class field not contain the actual user class loader in the 
first place. Should not that be the focus of the fix instead of pulling it out 
of the resourceManager only for one the methods?
   > 
   
   @afedulov That sounds reasonable. I will investigate more on this and try to 
figure out the reason. Meanwhile do you think a more targeted fix would be a 
better approach here. Something that is confined to 
[StreamingJobGraphGenerator.prevalidate](https://github.com/apache/flink/blob/4b306699811af0b6ff2cb862914adfda56345996/flink-streaming-java/src/main/java/org/apache/flink/streaming/api/graph/StreamingJobGraphGenerator.java#L502)
 method level by removing the class loading happening 
[here](https://github.com/apache/flink/blob/4b306699811af0b6ff2cb862914adfda56345996/flink-streaming-java/src/main/java/org/apache/flink/streaming/api/graph/StreamingJobGraphGenerator.java#L535).
 Due to this issue, the ADD JAR statement is causing a job failure when 
checkpointing is enabled, which is preventing to use this feature in 
production. 
   
   


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

Reply via email to