Hi, Stan, Thanks for the KIP. Looks good to me overall. Just one comment below.
uncleanable-partitions-count is per logDir, but uncleanable-bytes is not. Should we make them consistent? Jun On Wed, Aug 22, 2018 at 4:15 AM, Stanislav Kozlovski <stanis...@confluent.io > wrote: > Hi everybody, > > @Jason - I've updated the section. Thanks for the reminder > > I'm glad to say that the vote *has passed* with 3 binding votes (Jason, > Gwen, Harsha) and 6 non-binding votes (Dhruvil, Colin, Mickael, Manikumar, > Ray, Ted, Thomas). > > The PR is ready for review at https://github.com/apache/kafka/pull/5439 > > On Tue, Aug 21, 2018 at 4:55 PM Jason Gustafson <ja...@confluent.io> > wrote: > > > +1 Thanks for the KIP! I'd suggest mentioning the configurations that > were > > previously proposed in the rejected alternatives section. We may > reconsider > > them in the future. > > > > On Mon, Aug 13, 2018 at 9:48 AM, Dhruvil Shah <dhru...@confluent.io> > > wrote: > > > > > Thanks for the KIP, Stanislav! +1 (non-binding) > > > > > > - Dhruvil > > > > > > On Mon, Aug 13, 2018 at 9:39 AM Colin McCabe <cmcc...@apache.org> > wrote: > > > > > > > +1 (non-binding) > > > > > > > > best, > > > > Colin > > > > > > > > On Tue, Aug 7, 2018, at 04:19, Stanislav Kozlovski wrote: > > > > > Hey everybody, > > > > > I'm starting a vote on KIP-346 > > > > > < > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > 346+-+Improve+LogCleaner+behavior+on+error > > > > > > > > > > > > > > > -- > > > > > Best, > > > > > Stanislav > > > > > > > > > > > > -- > Best, > Stanislav >
