hi Stig We should run CI under Java 23 if we want to make 3.9 support Java 23 officially, so +1 to backport them
Best, Chia-Ping > Mickael Maison <mickael.mai...@gmail.com> 於 2025年3月28日 下午6:43 寫道: > > Hi, > > I'm also supportive of backporting this change to the 3.9 branch. > > Thanks, > Mickael > > > >> On Fri, Mar 28, 2025 at 10:16 AM Stig Rohde Døssing >> <stigdoess...@gmail.com> wrote: >> >> Pulling in a discussion from the PR thread: >> >>> There were other PRs that were part of Java 23 support generally, should >> these also be backported? https://issues.apache.org/jira/browse/KAFKA-17638 >> >> Greg, thanks for letting me know about these, I was not aware. >> >> I think most likely these changes aren't necessary to allow Kafka to run on >> Java 23. I believe upgrading Scala is necessary because the bundled ASM >> doesn't understand Java 23 bytecode, and the remaining changes seem to >> relate to either fixing compiler warnings or adding Java 23 to the build >> matrix. I think backporting these would be a fine idea, but I don't think >> they're blockers for using Kafka on Java 23 or 24. >> >> I'm happy to open PRs to backport these as well (though for the build >> matrix, we should use Java 24 now), assuming there's no strong opposition >> to that? >> >> Den fre. 28. mar. 2025 kl. 05.02 skrev Greg Harris >> <greg.har...@aiven.io.invalid>: >> >>> Hi All, >>> >>> I am still strongly in support of backporting this patch as I stated in the >>> earlier discussion thread. >>> >>> Thank you Stig, Anton, Danish, Mateusz, Manfred, Monica, Istvan, Vincent, >>> Clement, Anshu, Foivos, Drakgoku, Severin, Ozan, Georgios, and Guillaume, >>> members of our extended community who have commented to discuss the >>> impact and their support for this patch. >>> I inevitably missed some people, such as everyone who has been encountering >>> and silently working around the problem just from reading these threads and >>> GitHub issues. >>> >>> Through our inaction, we have obligated several users to take actions to >>> work around us: >>> * The Trino project dropped two of their products [1, 2] >>> * The Quarkus project disabled some tests [3] >>> * Quarkus project users downgraded their java version [4] >>> * Spring Boot users have installed the system property workaround [5] >>> >>> In my opinion this is unacceptable, and it's time for us to fix this. >>> >>> Thank you Stig for restarting the conversation! >>> Greg >>> >>> [1] https://github.com/trinodb/trino/issues/24419 >>> [2] https://github.com/trinodb/trino/issues/24417 >>> [3] https://github.com/quarkusio/quarkus/pull/43543 >>> [4] >>> >>> https://github.com/anshupitlia/product-information-system/commit/2c3a8dbd974dce0273f74969ec64b661abafef62 >>> [5] >>> >>> https://github.com/vividus-framework/vividus-build-system/commit/53b7016a0d3b0ba04a23b4b1892e1cf7f62ba0a5 >>> >>>> On Thu, Mar 27, 2025 at 7:39 PM Luke Chen <show...@gmail.com> wrote: >>> >>>> Hi Stig, >>>> >>>> Thanks for bringing this to us. >>>> I'm +1 for backporting to 3.9 branch since there's no workaround for Java >>>> 24. >>>> >>>> Thanks. >>>> Luke >>>> >>>> >>>> >>>> On Tue, Mar 18, 2025 at 1:14 AM Stig Rohde Døssing < >>> stigdoess...@gmail.com >>>>> >>>> wrote: >>>> >>>>> Thanks Ismail, >>>>> >>>>> I've opened https://github.com/apache/kafka/pull/19221 just to get any >>>>> test >>>>> failures out of the way in case it is decided to do this backport. >>>>> >>>>> I'm hoping people will weigh in with their concerns in this thread if >>>> they >>>>> don't like the idea of backporting this change. >>>>> >>>>> Den man. 17. mar. 2025 kl. 16.43 skrev Ismael Juma <m...@ismaeljuma.com >>>> : >>>>> >>>>>> Hi Stig, >>>>>> >>>>>> Kafka 4.0 is likely to be released in a day or two. Even so, I think >>> it >>>>>> makes sense to revive the backporting thread given the lack of >>>> workaround >>>>>> for Java 24. >>>>>> >>>>>> Ismael >>>>>> >>>>>> On Mon, Mar 17, 2025 at 7:44 AM Stig Rohde Døssing < >>>>> stigdoess...@gmail.com >>>>>>> >>>>>> wrote: >>>>>> >>>>>>> Hi, >>>>>>> >>>>>>> Some months ago, a reflective shim was added in >>>>>>> https://issues.apache.org/jira/browse/KAFKA-17078, in order to >>>> support >>>>>>> running Kafka with SASL on JDKs that no longer support the security >>>>>>> manager. >>>>>>> >>>>>>> This shim was added only to Kafka 4.0, but backporting was >>> discussed >>>> in >>>>>>> https://lists.apache.org/thread/vl43q9wqq4xs67xx61f0t0850y2b037o. >>>>> There >>>>>>> was >>>>>>> no clear consensus for or against backporting, but it ended up not >>>>>>> happening. At the time, users could work around the issue by >>> enabling >>>>> the >>>>>>> Security Manager again via a command-line flag. >>>>>>> >>>>>>> Java 24, which is planned to release tomorrow, no longer has this >>>>>>> workaround available. >>>>>>> >>>>>>> This leaves users running Java 23 (I am one) in a slightly >>>>> uncomfortable >>>>>>> spot. >>>>>>> >>>>>>> If Kafka releases 4.0 in the next month, we can rush to upgrade to >>>>> that, >>>>>>> and hope that the first release has no regressions. >>>>>>> >>>>>>> Otherwise, we will need to downgrade back to Java 21, since staying >>>> on >>>>> 23 >>>>>>> isn't a good idea past Oracle's quarterly security update in April >>>> (see >>>>>>> https://www.oracle.com/security-alerts/), which will include >>> patches >>>>>> that >>>>>>> won't be released for Java 23. >>>>>>> >>>>>>> Would there be strong objections to attempting a backport of this >>>> shim >>>>>> to a >>>>>>> 3.9.x release? >>>>>>> >>>>>> >>>>> >>>> >>>