CRZbulabula commented on a change in pull request #5287:
URL: https://github.com/apache/iotdb/pull/5287#discussion_r830544583



##########
File path: thrift-confignode/src/main/thrift/confignode.thrift
##########
@@ -21,7 +21,20 @@ include "rpc.thrift"
 namespace java org.apache.iotdb.confignode.rpc.thrift
 namespace py iotdb.thrift.confignode
 
-struct SetStorageGroupoReq {
+struct EndPoint {
+  1: required string ip
+  2: required i32 port
+}
+
+// The return status code and message in each response.
+struct TSStatus {

Review comment:
       The TSStatus is already defined in org.apache.iotdb.service.rpc.thrift. 
It can be just included. If there is a necessity to define a different Status 
code for ConfigNode, it should be renamed.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to