HappenLee commented on code in PR #64441:
URL: https://github.com/apache/doris/pull/64441#discussion_r3400665022
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/processor/post/materialize/LazyMaterializeTopN.java:
##########
@@ -162,6 +162,17 @@ private Plan doComputeTopN(PhysicalTopN<? extends Plan>
topN, CascadesContext ct
HashSet<SlotReference> rowIdSet = new HashSet<>();
StatementContext threadStatementContext =
StatementScopeIdGenerator.getStatementContext();
for (Relation relation : relationToLazySlotMap.keySet()) {
+ // TopN lazy materialization relies on BE adding a
GLOBAL_ROWID_COL to the
+ // tablet schema. When light_schema_change=false, the table
columns have
Review Comment:
@morrySnow Light schema change was introduced in doris version 1.2.
Legacy tables are rare, and this is not the default behavior. Simply avoid
this optimization.
--
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]
