liaoxin01 commented on code in PR #37903:
URL: https://github.com/apache/doris/pull/37903#discussion_r1679346462


##########
be/src/http/action/stream_load.cpp:
##########
@@ -705,8 +706,15 @@ Status StreamLoadAction::_data_saved_path(HttpRequest* 
req, std::string* file_pa
 
 void 
StreamLoadAction::_save_stream_load_record(std::shared_ptr<StreamLoadContext> 
ctx,
                                                 const std::string& str) {
-    auto stream_load_recorder =
-            
ExecEnv::GetInstance()->storage_engine().to_local().get_stream_load_recorder();
+    std::shared_ptr<StreamLoadRecorder> stream_load_recorder;
+    if (config::is_cloud_mode()) {
+        stream_load_recorder =
+                
ExecEnv::GetInstance()->storage_engine().to_cloud().get_stream_load_recorder();

Review Comment:
   The get_stream_load_recorder has been implemented in the base class, 
to_cloud or to_local is not necessary?



-- 
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: commits-unsubscr...@doris.apache.org

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

Reply via email to