[GitHub] [kafka-site] guozhangwang commented on pull request #304: Add chia7712 to committers
guozhangwang commented on pull request #304: URL: https://github.com/apache/kafka-site/pull/304#issuecomment-711025894 I missed that file. You are good to go. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [VOTE] KIP-516: Topic Identifiers
+1 non-binding. Thanks! On Sat, Oct 17, 2020 at 7:55 AM David Jacot 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 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 > > > wrote: > > > > > Hi Justine, > > > Thanks for the KIP, +1 (non-binding) > > > > > > On Thu, Oct 15, 2020 at 10:48 PM Lucas Bradstreet > > > 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 wrote: > > > > > > > > > Hi, Justine, > > > > > > > > > > Thanks for the updated KIP. +1 from me. > > > > > > > > > > Jun > > > > > > > > > > On Tue, Oct 13, 2020 at 2:38 PM Jun Rao 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 > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > > >
[GitHub] [kafka-site] chia7712 commented on pull request #304: Add chia7712 to committers
chia7712 commented on pull request #304: URL: https://github.com/apache/kafka-site/pull/304#issuecomment-710767238 @guozhangwang thanks for reviews! > you'd also need to git add the image file as well. The image file is already included. Please correct me if I misunderstand your point. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [VOTE] KIP-516: Topic Identifiers
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 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 > wrote: > > > Hi Justine, > > Thanks for the KIP, +1 (non-binding) > > > > On Thu, Oct 15, 2020 at 10:48 PM Lucas Bradstreet > > 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 wrote: > > > > > > > Hi, Justine, > > > > > > > > Thanks for the updated KIP. +1 from me. > > > > > > > > Jun > > > > > > > > On Tue, Oct 13, 2020 at 2:38 PM Jun Rao 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 > > > > >> > > > > > >> > > > > > > > > > > > >
[GitHub] [kafka-site] guozhangwang commented on pull request #304: Add chia7712 to committers
guozhangwang commented on pull request #304: URL: https://github.com/apache/kafka-site/pull/304#issuecomment-710760998 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org