It's been about 6 weeks since I started the discussion, with no questions. I'm going to start a vote.
Thanks, and best regards! Randall On Thu, Mar 18, 2021 at 8:24 AM Dongjin Lee <dong...@apache.org> wrote: > Great! If this KIP is merged first, I will update the PR accordingly. > > Thanks, > Dongjin > > On Thu, Mar 18, 2021, 10:04 PM Randall Hauch <rha...@apache.org> wrote: > > > Thanks, Dongjin. > > > > We still have some time before this KIP might be approved, so I don’t > want > > to block any work on KIP-653. It’s fine for the KIP-653 PR to be merged > > first. I’ll just have to update my PR when this KIP passes. > > > > On Thu, Mar 18, 2021 at 1:30 AM Dongjin Lee <dong...@apache.org> wrote: > > > > > Hi Randall, > > > > > > I am +1 for this proposal. Sure, changing this setting manually is so > > > annoying. I think this proposal should be applied as soon as possible. > > > > > > However, I have a question: as you already know, the upgrade to > > Log4j2[^1] > > > was already passed but not merged yet. Which one would be the right > > working > > > path? Merge KIP-653 first and KIP-721 later, or merge KIP-721 first and > > > apply the change into KIP-653? > > > > > > Thanks, > > > Dongjin > > > > > > [^1]: > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-653%3A+Upgrade+log4j+to+log4j2 > > > > > > On Wed, Mar 17, 2021 at 6:31 AM Randall Hauch <rha...@apache.org> > wrote: > > > > > > > Hello all, > > > > > > > > I'd like to propose KIP-721 to change Connect's Log4J configuration > > that > > > we > > > > ship with AK. This KIP will enable by default Connect's valuable > > > connector > > > > log contexts, which was added as part of KIP-449 to include > connector- > > > and > > > > task-specific information to every log message output by the > connector, > > > its > > > > tasks, or the worker thread operating those components. > > > > > > > > The details are here: > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-721%3A+Enable+connector+log+contexts+in+Connect+Log4j+configuration > > > > > > > > The earlier KIP-449 (approved and implemented in AK 2.3.0) is here: > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-449%3A+Add+connector+contexts+to+Connect+worker+logs > > > > > > > > I look forward to your feedback! > > > > > > > > Best regards, > > > > > > > > Randall > > > > > > > > > > > > > -- > > > *Dongjin Lee* > > > > > > *A hitchhiker in the mathematical world.* > > > > > > > > > > > > *github: <http://goog_969573159/>github.com/dongjinleekr > > > <https://github.com/dongjinleekr>keybase: > > https://keybase.io/dongjinleekr > > > <https://keybase.io/dongjinleekr>linkedin: > > kr.linkedin.com/in/dongjinleekr > > > <https://kr.linkedin.com/in/dongjinleekr>speakerdeck: > > > speakerdeck.com/dongjin > > > <https://speakerdeck.com/dongjin>* > > > > > >
