[GitHub] [flink] zentol commented on a diff in pull request #22794: [FLINK-25002][build] Add java 17 add-opens/add-exports JVM arguments

["zentol (via GitHub)" <gi...@apache.org>]

zentol commented on code in PR #22794:
URL: https://github.com/apache/flink/pull/22794#discussion_r1231022699


##########
flink-end-to-end-tests/flink-queryable-state-test/pom.xml:
##########
@@ -107,6 +107,9 @@
 							<transformers>
 								<transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
 									<mainClass>org.apache.flink.streaming.tests.queryablestate.QsStateClient</mainClass>
+									<manifestEntries>
+										<Add-Opens>java.base/java.util</Add-Opens>
+									</manifestEntries>

Review Comment:
   This is a handy alternative in case you run a self-contained jar; you can encode the opens/exports in the jars manifest.
   Unfortunately this only works if the you use the `-jar` command; we can't use it for the distribution :cry: 



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