Skip to content

Commit 22e0c1c

Browse files
authored
Add PolarisDiagnostics field to TransactionalMetaStoreManagerImpl (#2361)
the ultimate goal is removing the PolarisCallContext parameter from every PolarisMetaStoreManager interface method, so we make steps towards reducing its usage first.
1 parent 96f1459 commit 22e0c1c

File tree

6 files changed

+114
-159
lines changed

6 files changed

+114
-159
lines changed

persistence/eclipselink/src/test/java/org/apache/polaris/extension/persistence/impl/eclipselink/PolarisEclipseLinkMetaStoreManagerTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ protected PolarisTestMetaStoreManager createPolarisTestMetaStoreManager() {
8888
new PolarisEclipseLinkMetaStoreSessionImpl(
8989
store, Mockito.mock(), realmContext, null, "polaris", RANDOM_SECRETS);
9090
TransactionalMetaStoreManagerImpl metaStoreManager =
91-
new TransactionalMetaStoreManagerImpl(clock);
91+
new TransactionalMetaStoreManagerImpl(clock, diagServices);
9292
PolarisCallContext callCtx = new PolarisCallContext(realmContext, session, diagServices);
9393
return new PolarisTestMetaStoreManager(metaStoreManager, callCtx);
9494
}

polaris-core/src/main/java/org/apache/polaris/core/persistence/LocalPolarisMetaStoreManagerFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ protected PrincipalSecretsGenerator secretsGenerator(
8989
* into the existing realm-based setup flow.
9090
*/
9191
protected PolarisMetaStoreManager createNewMetaStoreManager(Clock clock) {
92-
return new TransactionalMetaStoreManagerImpl(clock);
92+
return new TransactionalMetaStoreManagerImpl(clock, diagnostics);
9393
}
9494

9595
private void initializeForRealm(

0 commit comments

Comments
 (0)