tomscut commented on PR #5123: URL: https://github.com/apache/hadoop/pull/5123#issuecomment-1320881262
> > > > What do you think about moving just > > > > message RouterFederatedStateProto { > > > > map<string, int64> namespaceStateIds = 1; // Last seen state IDs for multiple namespaces. > > > > } > > > > > > @simbadzina Nice suggestion, moving RouterFederatedStateProto to hdfs.proto? > > @ZanderXu Thanks. I was initially thinking of putting it in ClientNamenodeProtocol.Proto, but hdfs.proto is a good location too. I agree > > > What do you think about moving just > > > > message RouterFederatedStateProto { > > > > map<string, int64> namespaceStateIds = 1; // Last seen state IDs for multiple namespaces. > > > > } > > > > > > @simbadzina Nice suggestion, moving RouterFederatedStateProto to hdfs.proto? > > @ZanderXu Thanks. I was initially thinking of putting it in ClientNamenodeProtocol.Proto, but hdfs.proto is a good location too. I think moving RouterFederatedStateProto to hdfs.proto is ok. @goiri Can you please give some advices. Thank you very much. -- 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. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org