morrySnow opened a new pull request, #61256: URL: https://github.com/apache/doris/pull/61256
## Background `ClusterNamespace` is a legacy artifact from the multi-tenancy era. In earlier versions, Apache Doris implemented multi-tenancy by prefixing database names with a cluster name using `:` as a separator. This functionality has since been completely removed, but the `ClusterNamespace` class and related legacy code remained. ## Changes - Remove all `ClusterNamespace.getNameFromFullName()` calls (~122 call sites), replacing each with its original argument - Remove all `import org.apache.doris.cluster.ClusterNamespace` statements - Delete `ClusterNamespace.java` - Clean up dead code left behind (no-op self-assignments, stale comments like "should be removed in 3.0", empty `removeClusterPrefix()` methods) - Fix import ordering to satisfy checkstyle ## Scope 60 files changed across: - Privilege/role management (Auth, Role, RoleManager, etc.) - Nereids commands (ShowCreateDatabase, ShowPartitions, etc.) - Ranger authorization (RangerDorisAccessController, RangerHiveAccessController) - Catalog/datasource (CatalogMgr, Database, InternalCatalog, etc.) - HTTP/REST handlers (BaseController, RestBaseController, etc.) - Backup/persistence (BackupHandler, DatabaseInfo, etc.) - Other modules (FrontendServiceImpl, ConnectContext, etc.) ## Testing - FE compilation: ✅ BUILD SUCCESS - UserAuthenticationTest: ✅ 6/6 tests pass -- 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]
