+1 (binding). Thanks for the written KIP! The approach lgtm. One minor thing: the name of "last epoch" maybe a bit misleading (although it is for internal usage only and will not be exposed to users) for future developers, how about rename it to "required_epoch" and if it is set to "-1" it means "not required, hence not checks"?
Guozhang On Tue, Apr 9, 2019 at 5:02 PM Jason Gustafson <ja...@confluent.io> wrote: > Bump (for Guozhang) > > On Mon, Apr 8, 2019 at 8:55 AM Jason Gustafson <ja...@confluent.io> wrote: > >> Hi All, >> >> I'd like to start a vote on KIP-360: >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-360%3A+Improve+handling+of+unknown+producer >> . >> >> +1 from me (duh) >> >> Thanks, >> Jason >> > -- -- Guozhang
