eric-maynard commented on code in PR #899:
URL: https://github.com/apache/polaris/pull/899#discussion_r1934561524
##########
polaris-core/src/main/java/org/apache/polaris/core/persistence/LocalPolarisMetaStoreManagerFactory.java:
##########
@@ -70,44 +70,43 @@ protected LocalPolarisMetaStoreManagerFactory(
protected abstract PolarisMetaStoreSession createMetaStoreSession(
@Nonnull StoreType store,
- @Nonnull RealmId realmId,
+ @Nonnull Realm realm,
@Nullable PolarisCredentialsBootstrap credentialsBootstrap,
@Nonnull PolarisDiagnostics diagnostics);
protected PrincipalSecretsGenerator secretsGenerator(
- RealmId realmId, @Nullable PolarisCredentialsBootstrap
credentialsBootstrap) {
+ Realm realm, @Nullable PolarisCredentialsBootstrap credentialsBootstrap)
{
if (credentialsBootstrap != null) {
- return PrincipalSecretsGenerator.bootstrap(realmId.id(),
credentialsBootstrap);
+ return PrincipalSecretsGenerator.bootstrap(realm.name(),
credentialsBootstrap);
} else {
return PrincipalSecretsGenerator.RANDOM_SECRETS;
}
}
- private void initializeForRealm(
- RealmId realmId, PolarisCredentialsBootstrap credentialsBootstrap) {
+ private void initializeForRealm(Realm realm, PolarisCredentialsBootstrap
credentialsBootstrap) {
final StoreType backingStore = createBackingStore(diagnostics);
sessionSupplierMap.put(
- realmId.id(),
- () -> createMetaStoreSession(backingStore, realmId,
credentialsBootstrap, diagnostics));
+ realm.name(),
+ () -> createMetaStoreSession(backingStore, realm,
credentialsBootstrap, diagnostics));
PolarisMetaStoreManager metaStoreManager =
- new PolarisMetaStoreManagerImpl(realmId, diagnostics,
configurationStore, clock);
- metaStoreManagerMap.put(realmId.id(), metaStoreManager);
+ new PolarisMetaStoreManagerImpl(realm, diagnostics,
configurationStore, clock);
+ metaStoreManagerMap.put(realm.name(), metaStoreManager);
Review Comment:
If it's a pretty big change we can defer it. My concern is the same as you
noted in another comment, making everything consistent.
--
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
