+1 non-binding. Thanks! On Sat, Oct 17, 2020 at 7:55 AM David Jacot <david.ja...@gmail.com> wrote:
> Hi Justine, > > Thanks for the KIP! This is a great and long awaited improvement. > > +1 (binding) > > Best, > David > > Le ven. 16 oct. 2020 à 17:36, Rajini Sivaram <rajinisiva...@gmail.com> a > écrit : > > > Hi Justine, > > > > +1 (binding) > > > > Thanks for all the work you put into this KIP! > > > > btw, there is a typo in the DeleteTopics Request/Response schema in the > > KIP, it says Metadata request. > > > > Regards, > > > > Rajini > > > > > > On Fri, Oct 16, 2020 at 4:06 PM Satish Duggana <satish.dugg...@gmail.com > > > > wrote: > > > > > Hi Justine, > > > Thanks for the KIP, +1 (non-binding) > > > > > > On Thu, Oct 15, 2020 at 10:48 PM Lucas Bradstreet <lu...@confluent.io> > > > wrote: > > > > > > > > Hi Justine, > > > > > > > > +1 (non-binding). Thanks for all your hard work on this KIP! > > > > > > > > Lucas > > > > > > > > On Wed, Oct 14, 2020 at 8:59 AM Jun Rao <j...@confluent.io> wrote: > > > > > > > > > Hi, Justine, > > > > > > > > > > Thanks for the updated KIP. +1 from me. > > > > > > > > > > Jun > > > > > > > > > > On Tue, Oct 13, 2020 at 2:38 PM Jun Rao <j...@confluent.io> wrote: > > > > > > > > > > > Hi, Justine, > > > > > > > > > > > > Thanks for starting the vote. Just a few minor comments. > > > > > > > > > > > > 1. It seems that we should remove the topic field from the > > > > > > StopReplicaResponse below? > > > > > > StopReplica Response (Version: 4) => error_code [topics] > > > > > > error_code => INT16 > > > > > > topics => topic topic_id* [partitions] > > > > > > > > > > > > 2. "After controller election, upon receiving the result, assign > > the > > > > > > metadata topic its unique topic ID". Will the UUID for the > metadata > > > topic > > > > > > be written to the metadata topic itself? > > > > > > > > > > > > 3. The vote request is designed to support multiple topics, each > of > > > them > > > > > > may require a different sentinel ID. Should we reserve more than > > one > > > > > > sentinel ID for future usage? > > > > > > > > > > > > 4. UUID.randomUUID(): Could we clarify whether this method > returns > > > any > > > > > > sentinel ID? Also, how do we expect the user to use it? > > > > > > > > > > > > Thanks, > > > > > > > > > > > > Jun > > > > > > > > > > > > On Mon, Oct 12, 2020 at 9:54 AM Justine Olshan < > > jols...@confluent.io > > > > > > > > > > wrote: > > > > > > > > > > > >> Hi all, > > > > > >> > > > > > >> After further discussion and changes to this KIP, I think we are > > > ready > > > > > to > > > > > >> restart this vote. > > > > > >> > > > > > >> Again, here is the KIP: > > > > > >> > > > > > >> > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-516%3A+Topic+Identifiers > > > > > >> > > > > > >> The discussion thread is here: > > > > > >> > > > > > >> > > > > > > > > > > > https://lists.apache.org/thread.html/7efa8cd169cadc7dc9cf86a7c0dbbab1836ddb5024d310fcebacf80c@%3Cdev.kafka.apache.org%3E > > > > > >> > > > > > >> Please take a look and vote if you have a chance. > > > > > >> > > > > > >> Thanks, > > > > > >> Justine > > > > > >> > > > > > >> On Tue, Sep 22, 2020 at 8:52 AM Justine Olshan < > > > jols...@confluent.io> > > > > > >> wrote: > > > > > >> > > > > > >> > Hi all, > > > > > >> > > > > > > >> > I'd like to call a vote on KIP-516: Topic Identifiers. Here is > > the > > > > > KIP: > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-516%3A+Topic+Identifiers > > > > > >> > > > > > > >> > The discussion thread is here: > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > > > https://lists.apache.org/thread.html/7efa8cd169cadc7dc9cf86a7c0dbbab1836ddb5024d310fcebacf80c@%3Cdev.kafka.apache.org%3E > > > > > >> > > > > > > >> > Please take a look and vote if you have a chance. > > > > > >> > > > > > > >> > Thank you, > > > > > >> > Justine > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > > >
