dimas-b commented on code in PR #3669:
URL: https://github.com/apache/polaris/pull/3669#discussion_r2766638355
##########
runtime/service/src/main/java/org/apache/polaris/service/catalog/common/CatalogHandler.java:
##########
@@ -59,43 +52,25 @@ public abstract class CatalogHandler {
// Initialized in the authorize methods.
protected PolarisResolutionManifest resolutionManifest = null;
- protected final ResolutionManifestFactory resolutionManifestFactory;
Review Comment:
Just an idea: If we convert fields to abstract getters and make
implementations of this class `@Immutable`, the `CatalogHandlerRuntime` object
will probably become unnecessary and impl. instances will have direct
builders... WDYT?
--
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]
