Add comments

apache · Oct 3, 2024 · 99c4c2f · 99c4c2f
1 parent 1eb9bdc
commit 99c4c2f
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/...-core/src/test/java/org/apache/polaris/service/storage/PolarisConfigurationStoreTest.java b/...-core/src/test/java/org/apache/polaris/service/storage/PolarisConfigurationStoreTest.java
@@ -41,6 +41,10 @@ public void testConfigsCanBeCastedFromString() {
 
     PolarisConfigurationStore store =
         new PolarisConfigurationStore() {
+          /**
+           * Ad-hoc configuration store implementation that just returns the stringified version of
+           * the config's default value
+           */
           @SuppressWarnings("unchecked")
           @Override
           public <T> @Nullable T getConfiguration(PolarisCallContext ctx, String configName) {
@@ -57,6 +61,8 @@ public void testConfigsCanBeCastedFromString() {
           }
         };
 
+    // Ensure that we can fetch all the configs and that the value is what we expect, which
+    // is the config's default value based on how we've implemented PolarisConfigurationStore above.
     for (PolarisConfiguration<?> c : configs) {
       Assertions.assertEquals(c.defaultValue, store.getConfiguration(null, c));
     }