924060929 commented on code in PR #68196:
URL: https://github.com/apache/doris/pull/68196#discussion_r4059577691
##########
fe/fe-core/src/main/java/org/apache/doris/datasource/CatalogMgr.java:
##########
@@ -1050,6 +1050,10 @@ public void addExternalPartitions(String catalogName,
String dbName, String tabl
}
HMSExternalTable hmsTable = (HMSExternalTable) table;
+ // The metastore mutation has already committed when this event is
handled. Fence the
+ // independent row-count cache even when the local partition cache
cannot represent the
+ // table and this method returns early.
+
Env.getCurrentEnv().getExtMetaCacheMgr().invalidateRowCountCache(hmsTable);
Review Comment:
Fixed in 8e1a4789a843caaf84392ffc44191b2f26d764f8. Add/drop/alter partition
event paths now publish a cache-only row-count fence before any fallible
database/table reload. The manager resolves cached table identity when
available, falls back to the cached DB scope when the table is cold, and to
catalog scope when the DB is cold. Added cold-table ignored-miss coverage for
all three paths and a manager-level DB fallback test; focused 36 tests and the
full FE build passed.
--
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]
