+1. A quick clarification question regarding the compatibility plan as for "The legacy values for `enable.idempotence` will be interpreted as follows by the new producer: true will mean required, false will mean off."
Right now "enable.idempotence" is defined as Type.BOOLEAN while we are likely to change it to Type.STRING, is that considered compatible? Guozhang On Thu, Sep 7, 2017 at 2:25 PM, Jason Gustafson <ja...@confluent.io> wrote: > +1. Thanks for the KIP. One nit: we could use int8 to represent the message > format version. That is how it is represented in the messages themselves. > > -Jason > > On Thu, Sep 7, 2017 at 1:51 PM, Apurva Mehta <apu...@confluent.io> wrote: > > > Hi, > > > > I'd like to start a vote for KIP-192: > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 192+%3A+Provide+cleaner+semantics+when+idempotence+is+enabled > > > > Thanks, > > Apurva > > > -- -- Guozhang