Oops, hit send too soon. I see that #15127 was already merged. So we should no longer be publishing :metadata as part of the clients artifacts, right?
thanks, Colin On Mon, Jan 8, 2024, at 11:42, Colin McCabe wrote: > Hi Apporv, > > Please remove the metadata module from any artifacts published for > clients. It is only used by the server. > > best, > Colin > > > On Sun, Jan 7, 2024, at 03:04, Apoorv Mittal 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 <[email protected]> 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 >>> > <[email protected]> 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 >>>
