zhiqiang-hhhh commented on code in PR #45414:
URL: https://github.com/apache/doris/pull/45414#discussion_r1884109800
##########
be/src/vec/exprs/vcompound_pred.h:
##########
@@ -342,10 +380,9 @@ class VCompoundPred : public VectorizedFnCall {
return (l_null & r_null) | (r_null & (r_null ^ a)) | (l_null & (l_null
^ b));
}
- bool _all_child_is_compound_and_not_const() const {
- return std::ranges::all_of(_children, [](const VExprSPtr& arg) -> bool
{
- return arg->is_compound_predicate() && !arg->is_constant();
- });
+ bool _all_child_is_not_const() const {
Review Comment:
As we use this method like `!_all_child_is_not_const`, and it is equal to
`_has_const_child`, so I think we should implement `_has_const_child`, which is
more easy to understand.
--
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
