Thanks, Mickael.
Am closing the voting now with 3 binding votes (from Chris, Greg and
Mickale) and 1 non-binding (from Federico).
On Wed, Aug 2, 2023 at 2:52 PM Mickael Maison
wrote:
> +1 (binding)
>
> Thanks for the KIP
>
> On Tue, Aug 1, 2023 at 1:26 PM Omnia Ibrahim
> wrote:
> >
> > Thanks f
+1 (binding)
Thanks for the KIP
On Tue, Aug 1, 2023 at 1:26 PM Omnia Ibrahim wrote:
>
> Thanks for the binding vote, Greg, We now need one extra binding vote to
> get this KIP accepted.
>
> On Tue, Jul 25, 2023 at 8:10 PM Greg Harris
> wrote:
>
> > Hey Omnia,
> >
> > Thanks for the KIP!
> >
> >
Thanks for the binding vote, Greg, We now need one extra binding vote to
get this KIP accepted.
On Tue, Jul 25, 2023 at 8:10 PM Greg Harris
wrote:
> Hey Omnia,
>
> Thanks for the KIP!
>
> I think that MM2 is responsible for providing an upgrade path for
> users, even if it isn't backwards-compat
Hey Omnia,
Thanks for the KIP!
I think that MM2 is responsible for providing an upgrade path for
users, even if it isn't backwards-compatible by default due to a
mistake.
The non-configuration-based strategies I could think of aren't viable
due to the danger of inferring the incorrect topic name,
Hi Chris, I updated the KIP to address your feedback. Thanks for the vote.
On Mon, Jul 24, 2023 at 1:30 PM Chris Egerton
wrote:
> Hi Omnia,
>
> I think there's a few clarifications that should still be made on the KIP,
> but assuming these are agreeable, I'm +1 (binding)
>
> - In the description
Hi Omnia,
I think there's a few clarifications that should still be made on the KIP,
but assuming these are agreeable, I'm +1 (binding)
- In the description for the
replication.policy.internal.topic.separator.enabled property (in the
"Public Interfaces" section), we should specify that it affects
+1 (non binding)
Thanks
Fede
On Sun, Jul 23, 2023 at 6:30 PM Omnia Ibrahim wrote:
>
> Hi everyone,
> I would like to open a vote for KIP-949. The proposal is here
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-949%3A+Add+flag+to+enable+the+usage+of+topic+separator+in+MM2+DefaultReplica