Hi all,
Thanks for the KIP. This is definitely a worthwhile feature. However, I am
a bit sceptical on the ZK part of the story. The 3.8 release is supposed to
be the last one supporting ZK so I don't really see how we could bring it
to ZK, knowing that we don't plan to do a 3.9 release (current
Hi,
I agree with Chia-Ping, I think we could drop the ZK variant
altogether, especially if this is not going to make it in 3.8.0.
Even if we end up needing a 3.9.0 release, I wouldn't write a bunch of
new ZooKeeper-related code in that release to delete it all right
after in 4.0.
Thanks,
Mickael
Hello!
I am closing this vote as ACCEPTED with 3 binding +1 (Luke, Chia-Ping and
Satish) and 1 non-binding +1 (Kamal) - thank you for the reviews!
Realistically, I don't think I have the bandwidth to get this in 3.8.0.
Due to this, I will mark tentatively the Zookeeper part for 3.9 if the
+1
Thanks Christo for addressing the review comments. We can update the
KIP for any minor comments/clarifications.
On Thu, 16 May 2024 at 15:21, Luke Chen wrote:
>
> Thanks Chia-Ping!
> Since ZK is going to be removed, I agree the KRaft part has higher priority.
> But if Christo or the
Thanks Chia-Ping!
Since ZK is going to be removed, I agree the KRaft part has higher priority.
But if Christo or the community contributor has spare time, it's good to
have ZK part, too!
Thanks.
Luke
On Thu, May 16, 2024 at 5:45 PM Chia-Ping Tsai wrote:
> +1 but I prefer to ship it to KRaft
+1 but I prefer to ship it to KRaft only.
I do concern that community have enough time to accept more feature in 3.8 :(
Best,
Chia-Ping
On 2024/05/14 15:20:50 Christo Lolov wrote:
> Heya!
>
> I would like to start a vote on KIP-950: Tiered Storage Disablement in
> order to catch the last
Hi Christo,
Thanks for the KIP and incorporating the review comments.
Please update the KIP with the latest details.
+1 (non-binding).
Thanks,
Kamal
On Wed, May 15, 2024 at 3:18 PM Luke Chen wrote:
> Hi Christo,
>
> In addition to the minor comments left in the discussion thread, it LGTM.
Hi Christo,
In addition to the minor comments left in the discussion thread, it LGTM.
+1 from me.
Thank you.
Luke
On Tue, May 14, 2024 at 11:21 PM Christo Lolov
wrote:
> Heya!
>
> I would like to start a vote on KIP-950: Tiered Storage Disablement in
> order to catch the last Kafka release
Heya!
I would like to start a vote on KIP-950: Tiered Storage Disablement in
order to catch the last Kafka release targeting Zookeeper -
https://cwiki.apache.org/confluence/display/KAFKA/KIP-950%3A++Tiered+Storage+Disablement
Best,
Christo