Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-05-09 Thread Randall Hauch
Thanks everyone! The vote has been open for about 9 days, and the KIP is adopted with +4 binding votes (Randall, Manikumar, Gwen, and Bill) and no -1 votes. Thanks also to Jeremy, Magesh, Chris, and Konstantine for their non-binding votes. On Thu, May 9, 2019 at 1:59 PM Bill Bejeck wrote: > +1 (

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-05-09 Thread Bill Bejeck
+1 (binding) Makes a lot of sense to me. -Bill On Thu, May 9, 2019 at 2:05 PM Gwen Shapira wrote: > +1 (binding) > Hell yeah! > > On Mon, Apr 29, 2019 at 3:34 PM Randall Hauch wrote: > > > I would like to start the vote for KIP-258: > > > > > > > https://cwiki.apache.org/confluence/display/KAF

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-05-09 Thread Gwen Shapira
+1 (binding) Hell yeah! On Mon, Apr 29, 2019 at 3:34 PM Randall Hauch wrote: > I would like to start the vote for KIP-258: > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-449%3A+Add+connector+contexts+to+Connect+worker+logs > > The KIP uses the Mapped Diagnostic Contexts (MDC) featur

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-05-09 Thread Manikumar
HI Randall, +1 (binding), Thanks for the KIP. Thanks, Manikumar On Wed, May 8, 2019 at 4:29 AM Randall Hauch wrote: > +1 (binding) > > On Mon, May 6, 2019 at 3:53 PM Konstantine Karantasis < > konstant...@confluent.io> wrote: > > > Great improvement for multi-tenancy. > > Thanks Randall! > > >

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-05-07 Thread Randall Hauch
+1 (binding) On Mon, May 6, 2019 at 3:53 PM Konstantine Karantasis < konstant...@confluent.io> wrote: > Great improvement for multi-tenancy. > Thanks Randall! > > +1 (non-binding) > > Konstantine > > On Tue, Apr 30, 2019 at 9:18 PM Chris Egerton wrote: > > > +1 (non-binding) > > > > Really looki

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-05-06 Thread Konstantine Karantasis
Great improvement for multi-tenancy. Thanks Randall! +1 (non-binding) Konstantine On Tue, Apr 30, 2019 at 9:18 PM Chris Egerton wrote: > +1 (non-binding) > > Really looking forward to this. Thanks, Randall! > > On Tue, Apr 30, 2019, 20:47 Magesh Nandakumar > wrote: > > > This will make connec

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-04-30 Thread Chris Egerton
+1 (non-binding) Really looking forward to this. Thanks, Randall! On Tue, Apr 30, 2019, 20:47 Magesh Nandakumar wrote: > This will make connect debugging so much easier. Thanks a lot for driving > this Randall. > > +1 (non-binding) > > Thanks, > Magesh > > On Tue, Apr 30, 2019 at 7:19 PM Jeremy

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-04-30 Thread Magesh Nandakumar
This will make connect debugging so much easier. Thanks a lot for driving this Randall. +1 (non-binding) Thanks, Magesh On Tue, Apr 30, 2019 at 7:19 PM Jeremy Custenborder wrote: > +1 non binding > > On Mon, Apr 29, 2019 at 5:34 PM Randall Hauch wrote: > > > > I would like to start the vote f

Re: [VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-04-30 Thread Jeremy Custenborder
+1 non binding On Mon, Apr 29, 2019 at 5:34 PM Randall Hauch wrote: > > I would like to start the vote for KIP-258: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-449%3A+Add+connector+contexts+to+Connect+worker+logs > > The KIP uses the Mapped Diagnostic Contexts (MDC) feature of SLF4J

[VOT] KIP-449: Add connector contexts to Connect worker logs (vote thread)

2019-04-29 Thread Randall Hauch
I would like to start the vote for KIP-258: https://cwiki.apache.org/confluence/display/KAFKA/KIP-449%3A+Add+connector+contexts+to+Connect+worker+logs The KIP uses the Mapped Diagnostic Contexts (MDC) feature of SLF4J API to add more context to log messages from within Connect workers and connect