+1
On Mon, Apr 11, 2016 at 10:42 AM, Grant Henke <ghe...@cloudera.com> wrote: > Based on the discussion in the previous vote thread > <http://search-hadoop.com/m/uyzND1xlaiU10QlYX&subj=+VOTE+KIP+4+Metadata+Schema> > I also would like to include a behavior change to the MetadataResponse. I > have update the wiki > <https://cwiki.apache.org/confluence/display/KAFKA/KIP-4+-+Command+line+and+centralized+administrative+operations#KIP-4-Commandlineandcentralizedadministrativeoperations-MetadataSchema> > and pull request <https://github.com/apache/kafka/pull/1095> to include > this change. > > The change as described on the wiki is: > >> The behavior of the replicas and isr arrays will be changed in order to >> support the admin tools, and better represent the state of the cluster: >> >> - In version 0, if a broker is down the replicas and isr array will >> omit the brokers entry and add a REPLICA_NOT_AVAILABLE error code. >> - In version 1, no error code will be set and a the broker id will be >> included in the replicas and isr array. >> - Note: A user can still detect if the replica is not available, by >> checking if the broker is in the returned broker list. >> >> > > Being optimistic that this doesn't require to much discussion, I would like > to re-start the voting process on this thread. If more discussion is > needed, please don't hesitate to bring it up here. > > Ismael, Gwen, Guozhang could you please review and revote based on the > changes. > > Thank you, > Grant > > On Sat, Apr 9, 2016 at 1:03 PM, Guozhang Wang <wangg...@gmail.com> wrote: > >> +1 >> >> On Fri, Apr 8, 2016 at 4:36 PM, Gwen Shapira <g...@confluent.io> wrote: >> >> > +1 >> > >> > On Fri, Apr 8, 2016 at 2:41 PM, Grant Henke <ghe...@cloudera.com> wrote: >> > >> > > I would like to re-initiate the voting process for the "KIP-4 Metadata >> > > Schema changes". This is not a vote for all of KIP-4, but specifically >> > for >> > > the metadata changes. I have included the exact changes below for >> > clarity: >> > > > >> > > > Metadata Request (version 1) >> > > > >> > > > >> > > > >> > > > MetadataRequest => [topics] >> > > > >> > > > Stays the same as version 0 however behavior changes. >> > > > In version 0 there was no way to request no topics, and and empty >> list >> > > > signified all topics. >> > > > In version 1 a null topics list (size -1 on the wire) will indicate >> > that >> > > a >> > > > user wants *ALL* topic metadata. Compared to an empty list (size 0) >> > which >> > > > indicates metadata for *NO* topics should be returned. >> > > > Metadata Response (version 1) >> > > > >> > > > >> > > > >> > > > MetadataResponse => [brokers] controllerId [topic_metadata] >> > > > brokers => node_id host port rack >> > > > node_id => INT32 >> > > > host => STRING >> > > > port => INT32 >> > > > rack => NULLABLE_STRING >> > > > controllerId => INT32 >> > > > topic_metadata => topic_error_code topic is_internal >> > > [partition_metadata] >> > > > topic_error_code => INT16 >> > > > topic => STRING >> > > > is_internal => BOOLEAN >> > > > partition_metadata => partition_error_code partition_id leader >> > > [replicas] [isr] >> > > > partition_error_code => INT16 >> > > > partition_id => INT32 >> > > > leader => INT32 >> > > > replicas => INT32 >> > > > isr => INT32 >> > > > >> > > > Adds rack, controller_id, and is_internal to the version 0 response. >> > > > >> > > >> > > The KIP is available here for reference (linked to the Metadata schema >> > > section): >> > > * >> > > >> > >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-4+-+Command+line+and+centralized+administrative+operations#KIP-4-Commandlineandcentralizedadministrativeoperations-MetadataSchema >> > > < >> > > >> > >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-4+-+Command+line+and+centralized+administrative+operations#KIP-4-Commandlineandcentralizedadministrativeoperations-MetadataSchema >> > > >* >> > > >> > > A pull request is available implementing the proposed changes here: >> > > https://github.com/apache/kafka/pull/1095 >> > > >> > > Here are some links to past discussions on the mailing list: >> > > >> http://search-hadoop.com/m/uyzND1pd4T52H1m0u1&subj=Re+KIP+4+Wiki+Update >> > > >> > > >> > >> http://search-hadoop.com/m/uyzND1J2IXeSNXAT&subj=Metadata+and+ACLs+wire+protocol+review+KIP+4+ >> > > >> > > Here is the previous vote discussion (please take a look and discuss >> > > there): >> > > >> > > >> > >> http://search-hadoop.com/m/uyzND1xlaiU10QlYX&subj=+VOTE+KIP+4+Metadata+Schema >> > > >> > > Thank you, >> > > Grant >> > > -- >> > > Grant Henke >> > > Software Engineer | Cloudera >> > > gr...@cloudera.com | twitter.com/gchenke | linkedin.com/in/granthenke >> > > >> > >> >> >> >> -- >> -- Guozhang >> > > > > -- > Grant Henke > Software Engineer | Cloudera > gr...@cloudera.com | twitter.com/gchenke | linkedin.com/in/granthenke