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

Reply via email to