github-actions[bot] commented on code in PR #27596:
URL: https://github.com/apache/doris/pull/27596#discussion_r1405629523


##########
be/src/pipeline/exec/exchange_sink_operator.cpp:
##########
@@ -518,6 +518,16 @@ Status ExchangeSinkOperatorX::try_close(RuntimeState* 
state, Status exec_status)
     return final_st;
 }
 
+std::string ExchangeSinkLocalState::debug_string(int indentation_level) const {

Review Comment:
   warning: method 'debug_string' can be made static 
[readability-convert-member-functions-to-static]
   
   ```suggestion
   std::string ExchangeSinkLocalState::debug_string(int indentation_level) {
   ```
   
   be/src/pipeline/exec/exchange_sink_operator.h:188:
   ```diff
   -     std::string debug_string(int indentation_level) const override;
   +     static std::string debug_string(int indentation_level) override;
   ```
   



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to