zentol commented on PR #21532: URL: https://github.com/apache/flink/pull/21532#issuecomment-1385386814
> If you have any concerns, could you help me understand them? It boils down to not knowing how exactly FLINK-25002 will be realized and how much exactly this change enables. IIRC basically nothing worked without the described add-opens/add-exports, so AFAIK it will be unavoidable that we setup opens/exports in some form, be it either on the command-line or through reflection hacks at runtime (urgh). Whether this change here is useful depends on that. Additionally I'm just sceptical around enabling one particular class of applications while ignoring others (e.g., anything where the ClosureCleaner does some illegal access on some non-String type). I'd just like a more complete solution or even _vision_ of how we address this in the end before starting a whack-a-mole kind of effort. -- 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]
