nicoloboschi commented on code in PR #15264: URL: https://github.com/apache/pulsar/pull/15264#discussion_r864825795
########## README.md: ########## @@ -83,10 +83,35 @@ components in the Pulsar ecosystem, including connectors, adapters, and other la - [Pulsar CI](https://github.com/apache/pulsar-test-infra) +## Pulsar Runtime Java Version Recommendation + +- pulsar ver >= 2.11 Review Comment: 2.11 is not released. I believe it is better to keep the this table only up to 2.10.0, then update it when 2.11 will be publicly released ########## README.md: ########## @@ -83,10 +83,35 @@ components in the Pulsar ecosystem, including connectors, adapters, and other la - [Pulsar CI](https://github.com/apache/pulsar-test-infra) +## Pulsar Runtime Java Version Recommendation Review Comment: I believe we have to write much clear that the Pulsar IO and Functions will run on jdk17 at runtime. Here and in the website ########## README.md: ########## @@ -136,7 +161,8 @@ Check https://pulsar.apache.org for documentation and examples. Docker images must be built with Java 8 for `branch-2.7` or previous branches because of [issue 8445](https://github.com/apache/pulsar/issues/8445). -Java 11 is the recommended JDK version in `master`/`branch-2.8`. +Java 11 is the recommended JDK version in `branch-2.8`, and Review Comment: what about branch-2.9 and 2.10 ? -- 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]
