Fix RockDB config properties (#7942)

* Remove obsolete and unused RockDB config classes. * Update docs to use the new RockDB path property name.
projectnessie · Jan 11, 2024 · 6940fe7 · 6940fe7
1 parent f17fa9c
commit 6940fe7
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 35 deletions.
diff --git a/servers/quarkus-cli/src/main/resources/application.properties b/servers/quarkus-cli/src/main/resources/application.properties
@@ -51,7 +51,7 @@ quarkus.datasource.db-kind=postgresql
 quarkus.datasource.devservices.enabled=false
 
 ## RocksDB version store specific configuration
-#nessie.version.store.rocks.db-path=nessie-rocksdb
+#nessie.version.store.persist.rocks.database-path=nessie-rocksdb
 
 ## MongoDB version store specific configuration
 quarkus.mongodb.database=nessie

diff --git a/...ers/quarkus-common/src/main/java/org/projectnessie/quarkus/config/VersionStoreConfig.java b/...ers/quarkus-common/src/main/java/org/projectnessie/quarkus/config/VersionStoreConfig.java
@@ -47,12 +47,4 @@ enum VersionStoreType {
   @WithName("events.enable")
   @WithDefault("true")
   boolean isEventsEnabled();
-
-  @StaticInitSafe
-  @ConfigMapping(prefix = "nessie.version.store.rocks")
-  interface RocksVersionStoreConfig {
-    @WithName("db-path")
-    @WithDefault("/tmp/nessie-rocksdb")
-    String getDbPath();
-  }
 }
diff --git a/servers/quarkus-server/src/main/resources/application.properties b/servers/quarkus-server/src/main/resources/application.properties
@@ -96,7 +96,7 @@ quarkus.datasource.db-kind=postgresql
 quarkus.datasource.devservices.enabled=false
 
 ## RocksDB version store specific configuration
-#nessie.version.store.rocks.db-path=nessie-rocksdb
+#nessie.version.store.persist.rocks.database-path=nessie-rocksdb
 
 ## MongoDB version store specific configuration
 quarkus.mongodb.database=nessie

diff --git a/...main/java/org/projectnessie/quarkus/tests/profiles/RocksTestResourceLifecycleManager.java b/...main/java/org/projectnessie/quarkus/tests/profiles/RocksTestResourceLifecycleManager.java
diff --git a/site/docs/try/configuration.md b/site/docs/try/configuration.md
@@ -116,9 +116,9 @@ A complete set of JDBC configuration options can be found on [quarkus.io](https:
 
 When setting `nessie.version.store.type=ROCKSDB` which enables RocksDB as the version store used by the Nessie server, the following configurations are applicable in combination with `nessie.version.store.type`:
 
-| Property                             | Default values        | Type     | Description                                          |
-|--------------------------------------|-----------------------|----------|------------------------------------------------------|
-| `nessie.version.store.rocks.db-path` | `/tmp/nessie-rocksdb` | `String` | Sets RocksDB storage path, e.g: `/tmp/rocks-nessie`. |
+| Property                                           | Default values        | Type     | Description                                          |
+|----------------------------------------------------|-----------------------|----------|------------------------------------------------------|
+| `nessie.version.store.persist.rocks.database-path` | `/tmp/nessie-rocksdb` | `String` | Sets RocksDB storage path, e.g: `/tmp/rocks-nessie`. |
 
 #### Cassandra Version Store Settings