BiteTheDDDDt commented on code in PR #49803:
URL: https://github.com/apache/doris/pull/49803#discussion_r2041297258
##########
be/src/pipeline/exec/hashjoin_probe_operator.h:
##########
@@ -155,7 +155,8 @@ class HashJoinProbeOperatorX MOCK_REMOVE(final)
bool need_finalize_variant_column() const { return
_need_finalize_variant_column; }
bool is_lazy_materialized_column(int column_id) const {
- return _have_other_join_conjunct &&
!_other_conjunct_refer_column_ids.contains(column_id);
+ return _have_other_join_conjunct &&
Review Comment:
what if _have_other_join_conjunct is false and mark join is true
--
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
