From e6bbd03ea029c3100fdc5ffeb135b7c73ca89412 Mon Sep 17 00:00:00 2001
From: Kostas Zoumpatianos <kostas.zoumpatianos@fivetran.com>
Date: Fri, 29 Aug 2025 10:34:29 +0530
Subject: [PATCH] Added parameter and filter

---
 .../service/config/ProductionReadinessChecks.java        | 9 ++++++++-
 .../polaris/service/config/ReadinessConfiguration.java   | 8 ++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/runtime/service/src/main/java/org/apache/polaris/service/config/ProductionReadinessChecks.java b/runtime/service/src/main/java/org/apache/polaris/service/config/ProductionReadinessChecks.java
index 9f8ceb9a55..18ef5a97f7 100644
--- a/runtime/service/src/main/java/org/apache/polaris/service/config/ProductionReadinessChecks.java
+++ b/runtime/service/src/main/java/org/apache/polaris/service/config/ProductionReadinessChecks.java
@@ -74,7 +74,14 @@ public void warnOnFailedChecks(
       @Observes Startup event,
       Instance<ProductionReadinessCheck> checks,
       ReadinessConfiguration config) {
-    List<Error> errors = checks.stream().flatMap(check -> check.getErrors().stream()).toList();
+    List<Error> errors =
+        checks.stream()
+            .flatMap(check -> check.getErrors().stream())
+            .filter(
+                error ->
+                    config.ignoreOffendingProperties().stream()
+                        .noneMatch(prop -> prop.equalsIgnoreCase(error.offendingProperty())))
+            .toList();
     if (!errors.isEmpty()) {
       var utf8 = Charset.defaultCharset().equals(StandardCharsets.UTF_8);
       var warning = utf8 ? WARNING_SIGN_UTF_8 : WARNING_SIGN_PLAIN;
diff --git a/runtime/service/src/main/java/org/apache/polaris/service/config/ReadinessConfiguration.java b/runtime/service/src/main/java/org/apache/polaris/service/config/ReadinessConfiguration.java
index 01b4729335..d3c2494aa0 100644
--- a/runtime/service/src/main/java/org/apache/polaris/service/config/ReadinessConfiguration.java
+++ b/runtime/service/src/main/java/org/apache/polaris/service/config/ReadinessConfiguration.java
@@ -21,6 +21,7 @@
 import io.quarkus.runtime.annotations.StaticInitSafe;
 import io.smallrye.config.ConfigMapping;
 import io.smallrye.config.WithDefault;
+import java.util.Set;
 
 @StaticInitSafe
 @ConfigMapping(prefix = "polaris.readiness")
@@ -33,4 +34,11 @@ public interface ReadinessConfiguration {
    */
   @WithDefault("false")
   boolean ignoreSevereIssues();
+
+  /**
+   * Set of properties that, if found to be misconfigured, will be ignored when determining the
+   * production readiness.
+   */
+  @WithDefault("{}")
+  Set<String> ignoreOffendingProperties();
 }