yiguolei commented on code in PR #22315:
URL: https://github.com/apache/doris/pull/22315#discussion_r1276994333
##########
be/src/vec/exprs/vexpr.cpp:
##########
@@ -212,8 +213,11 @@ Status VExpr::create_expr(const TExprNode& expr_node,
VExprSPtr& expr) {
return Status::InternalError("Unknown expr node type: {}",
expr_node.node_type);
}
} catch (const Exception& e) {
- return Status::InternalError("create expr failed, TExprNode={},
reason={}",
-
apache::thrift::ThriftDebugString(expr_node), e.what());
+ if (e.code() == ErrorCode::INTERNAL_ERROR) {
+ return Status::InternalError("Create Expr failed because
{}\nTExprNode={}", e.what(),
+
apache::thrift::ThriftDebugString(expr_node));
+ }
+ return Status::Error(e.code(), "Create Expr failed because {}",
e.what());
Review Comment:
add thrift debug string, so we will know the expr
--
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
