lingbin commented on a change in pull request #2792: Reformat BE code 1/7 URL: https://github.com/apache/incubator-doris/pull/2792#discussion_r368279500
########## File path: be/src/agent/utils.cpp ########## @@ -59,26 +60,20 @@ using apache::thrift::transport::TTransportException; namespace doris { -MasterServerClient::MasterServerClient( - const TMasterInfo& master_info, - FrontendServiceClientCache* client_cache) : - _master_info(master_info), - _client_cache(client_cache) { -} +MasterServerClient::MasterServerClient(const TMasterInfo& master_info, + FrontendServiceClientCache* client_cache) : + _master_info(master_info), _client_cache(client_cache) {} -AgentStatus MasterServerClient::finish_task( - const TFinishTaskRequest& request, - TMasterResult* result) { +AgentStatus MasterServerClient::finish_task(const TFinishTaskRequest& request, + TMasterResult* result) { Status client_status; - FrontendServiceConnection client( - _client_cache, - _master_info.network_address, - config::thrift_rpc_timeout_ms, - &client_status); + FrontendServiceConnection client(_client_cache, _master_info.network_address, + config::thrift_rpc_timeout_ms, &client_status); Review comment: [mark 1], see comment at line 89. ---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org