morrySnow commented on code in PR #67783:
URL: https://github.com/apache/doris/pull/67783#discussion_r4023896367
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindRelation.java:
##########
@@ -625,9 +630,9 @@ private LogicalPlan buildMowTimeTravelUnion(LogicalOlapScan
baseScan, OlapTable
new TableScanParams(TableScanParams.INCREMENTAL_READ,
incrParams, Lists.newArrayList()));
LogicalPlan right = checkAndAddChangeScanFilter(binlogScan,
StreamScanType.MIN_DELTA, true);
- right = projectFromOriginSlots(right, visibleOutput);
+ right = projectFromUnboundSlots(new LogicalCheckPolicy<>(right),
visibleOutput);
Review Comment:
Thanks for the report. We are intentionally not mapping hidden origin
columns in this PR. A row policy should not be able to bind hidden columns in
the first place; the appropriate fix is in row-policy creation and validation
rather than in the time-travel branch mapping.
--
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]
