eric-maynard commented on code in PR #1806:
URL: https://github.com/apache/polaris/pull/1806#discussion_r2125678367
##########
polaris-core/src/main/java/org/apache/polaris/core/PolarisCallContext.java:
##########
@@ -22,13 +22,15 @@
import java.time.Clock;
import java.time.ZoneId;
import org.apache.polaris.core.config.PolarisConfigurationStore;
+import org.apache.polaris.core.context.CallContext;
+import org.apache.polaris.core.context.RealmContext;
import org.apache.polaris.core.persistence.BasePersistence;
/**
* The Call context is allocated each time a new REST request is processed. It
contains instances of
* low-level services required to process that request
*/
-public class PolarisCallContext {
+public class PolarisCallContext implements CallContext {
Review Comment:
Just curious, why not go the other way? It seems like it would be easier to
have CallContext extend PolarisCallContext, since it can immediately implement
all the methods just by delegating to the current PolarisCallContext
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
