MartijnVisser commented on PR #19764: URL: https://github.com/apache/flink/pull/19764#issuecomment-1129950651
> Because the issue may not be related to java 11 in the first place. The tests are unstable on Java 8 as well. Agreed. But even if they were stable on Java 8, I would still prefer to disable them on Java 11. We know the support isn't there and even if it runs OK, there's no guarantee that at some point in time it could break (because we might change something in Flink which then causes the test to fail). In the end, only the Pulsar maintainers can fix this problem and I think we should disable any Pulsar test on Java 11. > That's something different though; there we dealt with missing JDK dependencies. Actually I'm not sure why we disabled it in the first place, given that we supposedly fixed the dependency issue in the PR. According to the PR which introduced it https://github.com/apache/flink/pull/18815 the reason was `Disable the PulsarSourceOrderedE2ECase test on Java 11 for direct buffer memory error.` -- 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]
