dennishuo commented on code in PR #1070: URL: https://github.com/apache/polaris/pull/1070#discussion_r1972809547
########## polaris-core/src/main/java/org/apache/polaris/core/persistence/PolarisMetaStoreSession.java: ########## @@ -21,32 +21,19 @@ import jakarta.annotation.Nonnull; import jakarta.annotation.Nullable; import java.util.List; -import java.util.function.Function; -import java.util.function.Predicate; import java.util.function.Supplier; import org.apache.polaris.core.PolarisCallContext; import org.apache.polaris.core.entity.PolarisBaseEntity; -import org.apache.polaris.core.entity.PolarisChangeTrackingVersions; import org.apache.polaris.core.entity.PolarisEntitiesActiveKey; import org.apache.polaris.core.entity.PolarisEntityActiveRecord; import org.apache.polaris.core.entity.PolarisEntityCore; -import org.apache.polaris.core.entity.PolarisEntityId; -import org.apache.polaris.core.entity.PolarisEntityType; -import org.apache.polaris.core.entity.PolarisGrantRecord; -import org.apache.polaris.core.entity.PolarisPrincipalSecrets; -import org.apache.polaris.core.storage.PolarisStorageConfigurationInfo; -import org.apache.polaris.core.storage.PolarisStorageIntegration; /** - * Interface to the Polaris metadata store, allows to persist and retrieve all Polaris metadata like - * metadata for Polaris entities and metadata about grants between these entities which is the - * foundation of our role base access control model. - * - * <p>Note that APIs to the actual persistence store are very basic, often point read or write to - * the underlying data store. The goal is to make it really easy to back this using databases like - * Postgres or simpler KV store. + * Extends BasePersistence to express a more "transaction-oriented" control flow for backing stores Review Comment: Done -- 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]
