flyrain commented on code in PR #3340: URL: https://github.com/apache/polaris/pull/3340#discussion_r2666656662
########## runtime/server/src/main/java/org/apache/polaris/admintool/config/AdminToolProducers.java: ########## @@ -33,6 +35,15 @@ import org.apache.polaris.core.storage.PolarisStorageIntegration; import org.apache.polaris.core.storage.PolarisStorageIntegrationProvider; +/** + * CDI producer for admin tool mode. These beans provide minimal implementations for CLI operations + * and take precedence over service beans when running admin commands. + * + * <p>Note: This class uses @Alternative with @Priority to override ServiceProducers beans that + * would conflict (Clock, PolarisDiagnostics, MetaStoreManagerFactory). + */ +@Alternative +@Priority(100) // Higher priority than default to override ServiceProducers Review Comment: Removed it. The `ServiceProducers` has covered all of them. No need to have this one. -- 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]
