zy-kkk opened a new pull request, #57671:
URL: https://github.com/apache/doris/pull/57671
Problem Summary:
catalogs are incorrectly removed from the auto-refresh queue when:
- Creating a new catalog with `metadata_refresh_interval_sec`
- Altering catalog properties via `ALTER CATALOG SET PROPERTIES`
This causes auto-refresh to stop working even though
`metadata_refresh_interval_sec` is configured.
Root Cause:
CREATE/ALTER CATALOG
→ resetToUninitialized()
→ onClose()
→ CatalogIf.super.onClose()
→ removeFromRefreshMap(catalogId) ❌ Wrong!
Fix:
1. Remove `CatalogIf.super.onClose()` call from `ExternalCatalog.onClose()`
- Prevents wrongly removing catalog from refresh queue during CREATE/ALTER
2. Explicitly call `removeFromRefreshMap()` in `CatalogMgr.removeCatalog()`
- Ensures catalog is properly removed when actually dropping it
--
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]
