Hi Florin, Thanks for the KIP!
+1 (binding), Kind regards, Tom On Mon, Nov 22, 2021 at 6:51 AM Luke Chen <show...@gmail.com> wrote: > Hi Florin, > Thanks for the KIP. > > This KIP makes sense to me. Just a comment that the motivation section is > not clearly explain why this KIP is important. > I think John already mentioned a good motivation, which is to support "not > only UTF-8". > You should put that into the KIP, and of course if you have other thoughts, > please also add them into KIP. > > Also, in the "public interface" section, there are 3 "Default parsing > pattern", I think you should add 1 remaining case (false, false) to make it > complete. > > Otherwise, look good to me. > > Thank you. > Luke > > > On Sun, Nov 21, 2021 at 7:37 PM Florin Akermann <florin.akerm...@gmail.com > > > wrote: > > > Hi John, > > > > Thanks for the vote and feedback. > > > > The thought occurred to me too. > > > > Do I understand it correctly: the current version of the > > kafka-console-producer cannot be used for anything other than UTF-8 keys > > and values? > > (There is no other implementation of MessageReader other than the > > ConsoleProducer$LineMessageReader) > > In other words, currently users seem to only apply it with utf-8 strings > > for keys and values? > > This is why I figured I would not deviate from this assumption solely for > > the headers. > > > > I will happily raise another KIP / Jira if there is a need to specify > other > > formats / serializers for headers, keys and/or values. > > > > Thanks, > > Florin > > > > > > On Sat, 20 Nov 2021 at 19:34, John Roesler <vvcep...@apache.org> wrote: > > > > > Hi Florin, > > > > > > Thanks for the KIP! > > > > > > I think the assumption that header values are UTF-8 strings might not > > hold > > > up in the long run, but it seems like we can easily add a property > later > > to > > > specify the format. It seems like this scope is probably a handy > addition > > > on its own. > > > > > > I’m +1 (binding) > > > > > > Thanks, > > > John > > > > > > > > > On Fri, Nov 19, 2021, at 15:06, flo wrote: > > > > < > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-798%3A+Add+possibility+to+write+kafka+headers+in+Kafka+Console+Producer > > > > > > > > > >
