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

Reply via email to