eric-maynard commented on code in PR #1532: URL: https://github.com/apache/polaris/pull/1532#discussion_r2083010311
########## polaris-core/src/main/java/org/apache/polaris/core/config/FeatureConfiguration.java: ########## @@ -234,4 +233,38 @@ public static void enforceFeatureEnabledOrThrow( .description("If true, the policy-store endpoints are enabled") .defaultValue(true) .buildFeatureConfiguration(); + + public static final FeatureConfiguration<Boolean> ALLOW_SPECIFYING_FILE_IO_IMPL = + PolarisConfiguration.<Boolean>builder() + .key("ALLOW_SPECIFYING_FILE_IO_IMPL") + .description( + "Config key for whether to allow setting the FILE_IO_IMPL using catalog properties. " + + "Must only be enabled in dev/test environments, never in production systems.") + .defaultValue(false) + .buildFeatureConfiguration(); + + public static final FeatureConfiguration<Boolean> Review Comment: I have a hard time believing we can't incrementalize this change the evaluate the necessity of the individual components of it. We're going from this being in the default config for multiple releases to requiring multiple flags to enable it. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org