caiconghui commented on a change in pull request #5027:
URL: https://github.com/apache/incubator-doris/pull/5027#discussion_r538151204
##########
File path: gensrc/thrift/FrontendService.thrift
##########
@@ -654,6 +654,25 @@ struct TSnapshotLoaderReportRequest {
5: optional i32 total_num
}
+enum TFrontendPingFrontendStatusCode {
+ OK = 0,
+ FAILED = 1
+}
+
+struct TFrontendPingFrontendRequest {
+ 1: required i32 clusterId
+ 2: required string token
+}
+
+struct TBootstrapResult {
+ 1: required TFrontendPingFrontendStatusCode status
Review comment:
Yes, I am also not sure to reuse the TStatus or just like
```
enum TBrokerOperationStatusCode {
OK = 0;
END_OF_FILE = 301;
// user input error
NOT_AUTHORIZED = 401;
DUPLICATE_REQUEST = 402;
INVALID_INPUT_OFFSET = 403; // user input offset is invalid, is large
than file length
INVALID_INPUT_FILE_PATH = 404;
INVALID_ARGUMENT = 405;
// internal server error
FILE_NOT_FOUND = 501;
TARGET_STORAGE_SERVICE_ERROR = 502; // the target storage service error
OPERATION_NOT_SUPPORTED = 503; // the api is not implemented
}
```
define new status struct
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
