yiguolei commented on code in PR #66472:
URL: https://github.com/apache/doris/pull/66472#discussion_r3747557874


##########
be/src/exec/operator/scan_operator.cpp:
##########
@@ -580,8 +580,8 @@ Status 
ScanLocalStateBase::_normalize_function_filters(VExprContext* expr_ctx, S
         
RETURN_IF_ERROR(_should_push_down_function_filter(assert_cast<VectorizedFnCall*>(fn_expr),
                                                           expr_ctx, &val, 
&fn_ctx, temp_pdt));
         if (temp_pdt != PushDownType::UNACCEPTABLE) {
-            std::string col = slot->col_name();
-            _push_down_functions.emplace_back(opposite, col, fn_ctx, val);
+            int column_id = 
_parent->intermediate_row_desc().get_column_id(slot->id());

Review Comment:
   这个看之前file scan operator的逻辑,感觉不对啊?
   是不是继续保留get_column_id 这个方法,然后我们只改变内表的实现?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to