zentol opened a new pull request, #22738:
URL: https://github.com/apache/flink/pull/22738

   Enables `IgnoreUnrecognizedVMOptions` in all processes we start, so that we 
can later use add-opens/add-exports without having to worry about the actually 
used JDK.
   
   This option is _intentionally_ not set via `env.java.opts.all` because we 
really want this to be activated at all times, and not be disabled by users. In 
particular I'm worried about users copying their old configuration over to a 
new cluster, overriding whatever we have set as a new default.
   This isn't ideal though.
   
   In a way we could really use something like the parent-first `base` option, 
with a separate key for users to add additional items. But we already had 
issues with how parts of the system handle this option (FLINK-31913, 
FLINK-31916, FLINK-31915), so let's not add more fuel to the flames.


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

Reply via email to