morrySnow commented on code in PR #64441:
URL: https://github.com/apache/doris/pull/64441#discussion_r3400632498
##########
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:
When a table has light_schema_change=false, does that mean lazy
materialization cannot be used for any tables involved in the query?
--
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]
