morrySnow commented on code in PR #35436:
URL: https://github.com/apache/doris/pull/35436#discussion_r1615622913
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/StatementContext.java:
##########
@@ -149,6 +152,11 @@ public class StatementContext implements Closeable {
// Record table id mapping, the key is the hash code of union catalogId,
databaseId, tableId
// the value is the auto-increment id in the cascades context
private final Map<TableIdentifier, TableId> tableIdMapping = new
LinkedHashMap<>();
+ // Record the materialization statistics by id which is used for cost
estimation.
+ // Maybe return null, which means the id according statistics should calc
normally rather than getting
+ // form this map
+ // id maybe relation id or cteId or other type of id
+ private final Map<Pair<Id, Class<? extends Id>>, Statistics>
idToStatisticsMap = new LinkedHashMap<>();
Review Comment:
just support relation id now
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
