Re: [VOTE] KIP-784: Add top-level error code field to DescribeLogDirsResponse

2022-01-25 Thread Mickael Maison
Hi, Since it's a pretty minor KIP, I'll do +1 (binding) myself and close the vote. The vote has passed with: - 3 +1 (binding) votes from Tom, David and Mickael - 1 +1 (non binding) vote from Luke Thanks, Mickael On Thu, Dec 16, 2021 at 10:07 AM David Jacot wrote: > > +1 (binding). Thanks for

Re: [VOTE] KIP-784: Add top-level error code field to DescribeLogDirsResponse

2021-12-16 Thread David Jacot
+1 (binding). Thanks for the KIP! On Mon, Dec 13, 2021 at 11:14 AM Mickael Maison wrote: > > Bumping this thread another time. > > This is a very minor change to make DescribeLogDirsResponse consistent > with the other APIs. > Let me know if you have any feedback. > > Thanks, > Mickael > > On Mon

Re: [VOTE] KIP-784: Add top-level error code field to DescribeLogDirsResponse

2021-12-13 Thread Mickael Maison
Bumping this thread another time. This is a very minor change to make DescribeLogDirsResponse consistent with the other APIs. Let me know if you have any feedback. Thanks, Mickael On Mon, Nov 22, 2021 at 10:29 AM Tom Bentley wrote: > > Hi Mickael, > > It's pretty low value, but I think consiste

Re: [VOTE] KIP-784: Add top-level error code field to DescribeLogDirsResponse

2021-11-22 Thread Tom Bentley
Hi Mickael, It's pretty low value, but I think consistency is a useful trait, and it's easily achievable here. +1 (binding). Kind regards, Tom On Thu, Nov 18, 2021 at 2:56 PM Mickael Maison wrote: > Bumping this thread. > > Let me know if you have any feedback. > > Thanks, > Mickael > > On

Re: [VOTE] KIP-784: Add top-level error code field to DescribeLogDirsResponse

2021-11-18 Thread Mickael Maison
Bumping this thread. Let me know if you have any feedback. Thanks, Mickael On Wed, Oct 27, 2021 at 3:25 PM Luke Chen wrote: > > Hi Mickael, > Thanks for the KIP. > It's good to keep it consistent with others, to have top-level error field. > > + 1 (non-binding) > > Thank you. > Luke > > On Wed,

Re: [VOTE] KIP-784: Add top-level error code field to DescribeLogDirsResponse

2021-10-27 Thread Luke Chen
Hi Mickael, Thanks for the KIP. It's good to keep it consistent with others, to have top-level error field. + 1 (non-binding) Thank you. Luke On Wed, Oct 27, 2021 at 9:01 PM Mickael Maison wrote: > Hi all, > > I'd like to start the vote on this minor KIP. > > https://cwiki.apache.org/confluenc