yiguolei commented on code in PR #43281: URL: https://github.com/apache/doris/pull/43281#discussion_r1831135827
########## be/src/util/arrow/utils.cpp: ########## @@ -32,14 +32,15 @@ Status to_doris_status(const arrow::Status& status) { } } -arrow::Status to_arrow_status(const Status& status) { - if (status.ok()) { +arrow::Status to_arrow_status(const Status& status, const std::string& msg) { Review Comment: 不要这么改代码 如果你觉得status 里的error message 不对,那么就改变status 就行了,或者构造一个新的status传递进来 -- 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