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