csun5285 commented on code in PR #69812:
URL: https://github.com/apache/doris/pull/69812#discussion_r4410382971
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/VariantSubPathPruning.java:
##########
@@ -91,6 +92,14 @@ public class VariantSubPathPruning implements CustomRewriter
{
@Override
public Plan rewriteRoot(Plan plan, JobContext jobContext) {
+ // A changelog stores its Variant columns with every path in the
sparse column and maps a BEFORE image onto
+ // its AFTER column by whole column, so a changelog scan reads Variant
columns whole and sub-paths are taken
+ // above it. Its plan may union the changelog scan with a base table
scan, whose union pushdown needs every
+ // child to supply the same sub-path slots, so the whole statement
keeps whole-column reads.
+ if (plan.anyMatch(node -> node instanceof LogicalOlapScan
Review Comment:
ok, i will ban this case
--
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]
