Hi, we have fixed one memory leak in Kafka Streams, but there is still at least one missing in the code. I created https://issues.apache.org/jira/browse/KAFKA-16089 which is a blocker.
Cheers, Lucas On Sun, Jan 7, 2024 at 12:05 PM Apoorv Mittal <apoorvmitta...@gmail.com> wrote: > > Hi Colin, > Thanks for the response. The only reason for asking the question of > publishing the metadata is because that's present in previous client > releases. For more context, the description of PR > <https://github.com/apache/kafka/pull/15127> holds the details and waiting > for the confirmation there prior to the merge. > > Regards, > Apoorv Mittal > +44 7721681581 > > > On Fri, Jan 5, 2024 at 10:22 PM Colin McCabe <cmcc...@apache.org> wrote: > > > metadata is an internal gradle module. It is not used by clients. So I > > don't see why you would want to publish it (unless I'm misunderstanding > > something). > > > > best, > > Colin > > > > > > On Fri, Jan 5, 2024, at 10:05, Stanislav Kozlovski wrote: > > > Thanks for reporting the blockers, folks. Good job finding. > > > > > > I have one ask - can anybody with Gradle expertise help review this small > > > PR? https://github.com/apache/kafka/pull/15127 (+1, -1) > > > In particular, we are wondering whether we need to publish module > > metadata > > > as part of the gradle publishing process. > > > > > > > > > On Fri, Jan 5, 2024 at 3:56 PM Proven Provenzano > > > <pprovenz...@confluent.io.invalid> wrote: > > > > > >> We have potentially one more blocker > > >> https://issues.apache.org/jira/browse/KAFKA-16082 which might cause a > > data > > >> loss scenario with JBOD in KRaft. > > >> Initial analysis thought this is a problem and further review looks > > like it > > >> isn't but we are continuing to dig into the issue to ensure that it > > isn't. > > >> We would request feedback on the bug from anyone who is familiar with > > this > > >> code. > > >> > > >> --Proven > > >> > > > > > > > > > -- > > > Best, > > > Stanislav > >