wenzhenghu commented on code in PR #65126:
URL: https://github.com/apache/doris/pull/65126#discussion_r3541921312
##########
fe/fe-core/src/main/java/org/apache/doris/datasource/ExternalDatabase.java:
##########
@@ -432,15 +445,11 @@ public DatabaseProperty getDbProperties() {
public boolean isTableExist(String tableName) {
Review Comment:
Thank you, this is a very helpful observation.
I agree this corner case is real for the replay path when the names snapshot
has been invalidated but the object cache is still hot. I have addressed it in
this PR by adding a replay-only, cache-only fallback for mode 2: if the names
snapshot is cold, `getTableForReplay(String)` now performs a case-insensitive
scan over the current hot object-cache keys instead of reloading names.
This keeps the replay path non-blocking while covering the names-only
invalidation case. I also added a unit test for this scenario.
--
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]
