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

2023-06-16 Thread via GitHub
zentol commented on PR #22794: URL: https://github.com/apache/flink/pull/22794#issuecomment-1594624341 Yarn failure should be fixed; we were missing an `IgnoreUnrecognizedVMOptions` for yarn TMs. -- This is an automated message from the Apache Git Service. To respond to the message,

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

2023-06-16 Thread via GitHub
zentol commented on PR #22794: URL: https://github.com/apache/flink/pull/22794#issuecomment-1594606705 hmm the yarn tests seems to reliably get stuck now; investigating... It's quite odd because the tests pass just fine on the JDK 17 branch. :thinking: -- This is an automated message

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

2023-06-15 Thread via GitHub
zentol commented on PR #22794: URL: https://github.com/apache/flink/pull/22794#issuecomment-1593218659 > oh boy CI is broken pretty badly. I think this is the first time I'm running all this stuff on Java 8 eyes Fixed. Bit odd that surefire just hangs if the JVM crashes immediately