wenzhenghu commented on PR #65126: URL: https://github.com/apache/doris/pull/65126#issuecomment-4910817730
workbuddy + kimi 2.7-code： **核心结论** - 重构方向正确：`MetaCacheEntry` + `NameCacheValue` 拆分清晰，慢 I/O 移出 Caffeine 同步 load 路径，`loadLock`/`publishLock`/`generation` 三层并发协调严谨，replay 非阻塞。 - 最新提交 `142aa9f3afe` 已修复运行时 id 映射维护问题，`updateDatabaseCache`/`updateTableCache` 会始终回填 `dbIdToName`/`tableIdToName`。 - **建议在合入前处理的问题**：`ExternalCatalog.gsonPostProcess()` 当前仅新建空 `dbIdToName`，未从已持久化的 `dbNameToId` 反填，FE 重启后首次按 id 的 replay/查询可能返回空。建议补充反填逻辑。 其余为 minor：mode 2 + meta-names 映射的 key 不一致（历史遗留）、`buildMetaCache` 双字段判重竞态、`stats()` 统计口径混合、构造函数重载过多、`MetaCacheEntry` 二次 generation 检查为 test-only 路径。 **总体评价：Approve（建议合入，须处理 gsonPostProcess id 映射回填）**。 -- 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]
