fix for catalog config

Author: eric-maynard

polaris-core/src/main/java/org/apache/polaris/core/config/BehaviorChangeConfiguration.java

@@ -22,11 +22,11 @@
 
 /**
  * Internal configuration flags for non-feature behavior changes in Polaris. These flags control
- * subtle behavior adjustments and bug fixes, not user-facing catalog settings. They are intended
- * for internal use only, are inherently unstable, and may be removed at any time. When introducing
- * a new flag, consider the trade-off between maintenance burden and the risk of an unguarded
- * behavior change. Flags here are generally short-lived and should either be removed or promoted to
- * stable feature flags before the next release.
+ * subtle behavior adjustments and bug fixes, not user-facing settings. They are intended for
+ * internal use only, are inherently unstable, and may be removed at any time. When introducing a
+ * new flag, consider the trade-off between maintenance burden and the risk of an unguarded behavior
+ * change. Flags here are generally short-lived and should either be removed or promoted to stable
+ * feature flags before the next release.
  *
  * @param <T> The type of the configuration
  */
@@ -80,8 +80,8 @@ protected BehaviorChangeConfiguration(
           .key("ALLOW_NAMESPACE_CUSTOM_LOCATION")
           .catalogConfig("polaris.config.namespace-custom-location.enabled")
           .description(
-              "If set to true, allow namespaces with completely arbitrary locations. This should not affect" +
-                      " credential vending.")
+              "If set to true, allow namespaces with completely arbitrary locations. This should not affect"
+                  + " credential vending.")
           .defaultValue(false)
           .buildBehaviorChangeConfiguration();
 }

polaris-core/src/main/java/org/apache/polaris/core/config/PolarisConfiguration.java

@@ -209,10 +209,6 @@ public FeatureConfiguration<T> buildFeatureConfiguration() {
 
     public BehaviorChangeConfiguration<T> buildBehaviorChangeConfiguration() {
       validateOrThrow();
-      if (catalogConfig.isPresent() || catalogConfigUnsafe.isPresent()) {
-        throw new IllegalArgumentException(
-            "catalog configs are not valid for behavior change configs");
-      }
       BehaviorChangeConfiguration<T> config =
           new BehaviorChangeConfiguration<>(
               key, description, defaultValue, catalogConfig, catalogConfigUnsafe);

runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalog.java

@@ -677,7 +677,9 @@ public boolean setProperties(Namespace namespace, Map<String, String> properties
     }
     if (!realmConfig.getConfig(
         BehaviorChangeConfiguration.ALLOW_NAMESPACE_CUSTOM_LOCATION, catalogEntity)) {
-      validateNamespaceLocation(NamespaceEntity.of(entity), resolvedEntities);
+      if (properties.containsKey(PolarisEntityConstants.ENTITY_BASE_LOCATION)) {
+        validateNamespaceLocation(NamespaceEntity.of(entity), resolvedEntities);
+      }
     }
 
     List<PolarisEntity> parentPath = resolvedEntities.getRawFullPath();