mchades commented on code in PR #11209:
URL: https://github.com/apache/gravitino/pull/11209#discussion_r3298058723
##########
plugins/idp-basic/src/main/java/org/apache/gravitino/idp/storage/relational/IdpRelationalStorage.java:
##########
@@ -30,28 +31,26 @@
import org.apache.gravitino.storage.relational.session.SqlSessionFactoryHelper;
/** JDBC bootstrap for built-in IdP relational storage. */
-public final class IdpRelationalStorage {
+public final class IdpRelationalStorage implements Closeable {
private static final Map<JDBCBackendType, String> EMBEDDED_JDBC_DATABASE_MAP
=
ImmutableMap.of(JDBCBackendType.H2, H2Database.class.getCanonicalName());
- private static JDBCDatabase jdbcDatabase;
-
- private IdpRelationalStorage() {}
+ private JDBCDatabase jdbcDatabase;
/**
* Initializes the JDBC session factory and optional embedded database.
*
* @param config The server configuration.
*/
- public static void initialize(Config config) {
+ public IdpRelationalStorage(Config config) {
Review Comment:
Should this be a singleton?
--
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]
