0AyanamiRei commented on code in PR #64933:
URL: https://github.com/apache/doris/pull/64933#discussion_r3502761732


##########
be/src/exec/operator/schema_scan_operator.cpp:
##########
@@ -186,6 +187,8 @@ Status SchemaScanOperatorX::prepare(RuntimeState* state) {
         for (; j < columns_desc.size(); ++j) {
             if (boost::iequals(_dest_tuple_desc->slots()[i]->col_name(), 
columns_desc[j].name)) {
                 _slot_offsets[i] = j;
+                _common_scanner_param->required_columns.insert(

Review Comment:
   Added planner-path regression coverage in 
372436222407013a13e6312bfd574abb283af763. The new information_schema_p0 suite 
checks UPDATE_TIME-only, TABLE_NAME+UPDATE_TIME, TABLE_NAME-filtered 
projection, and a full-status contrast case so the schema scan tuple pruning 
path is covered beyond the hand-built FE RPC requiredColumns tests.



-- 
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