Makes sense. Looks like a good improvement. Thanks for including the evaluation in the proposal Dongjin.
+1 (binding) Konstantine On Wed, Jun 9, 2021 at 6:59 PM Dongjin Lee <dong...@apache.org> wrote: > Thanks Ismel, Tom and Ryanne, > > I am now updating the KIP about the further works. Sure, You won't be > disappointed. > > As of Present: > > - binding: +2 (Ismael, Tom) > - non-binding: +1 (Ryanne) > > Anyone else? > > Best, > Dongjin > > On Thu, Jun 10, 2021 at 2:03 AM Tom Bentley <tbent...@redhat.com> wrote: > > > Hi Dongjin, > > > > Thanks for the KIP, +1 (binding). > > > > Kind regards, > > > > Tom > > > > On Wed, Jun 9, 2021 at 5:16 PM Ismael Juma <ism...@juma.me.uk> wrote: > > > > > I'm +1 on the proposed change. As I stated in the discuss thread, I > don't > > > think we should rule out the buffer size config, but we could list that > > as > > > future work vs rejected alternatives. > > > > > > Ismael > > > > > > On Sat, Jun 5, 2021 at 2:37 PM Dongjin Lee <dong...@apache.org> wrote: > > > > > > > Hi all, > > > > > > > > I'd like to open a voting thread for KIP-390: Support Compression > Level > > > > (rebooted): > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-390%3A+Support+Compression+Level > > > > > > > > Best, > > > > Dongjin > > > > > > > > -- > > > > *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>* > > > > > > > > > > > > -- > *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>* >
