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

   This PR sets of all required add-opens/add-exports statements for Maven, 
surefire, test-internal JVMs and flink-dist.
   
   If a module requires such a statement for surefire/testing then it must 
define a list of statements in the `surefire.module.config` property of the 
module's pom. This property is set on the test JVMs via surefire, but is also 
exposed as a system property to the test itself such that utils like the 
`TestJvmProcess` can forward them.
   I documented a hint for each statement as to why it is required.
   
   flink-dist is configured separately via `flink-conf.yaml`.
   
   The required statements were purely determined by running tests and checking 
what fails.
   "Completeness" was validated on my personal 
[CI](https://dev.azure.com/chesnay/flink/_build/results?buildId=3606&view=results);
 we may not be covering all tests since some only run on the apache branches. 
We'll just amend missing ones in the future if necessary.


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