Fixed production readiness tests

Author: fivetran-kostaszoumpatianos

runtime/service/src/main/java/org/apache/polaris/service/config/ProductionReadinessChecks.java

@@ -116,11 +116,16 @@ public void warnOnFailedChecks(
 
   @Produces
   public ProductionReadinessCheck checkMetricTags(MetricsConfiguration config) {
-    if (config.userPrincipalTag().enableInApiMetrics()) {
+    if (config.userPrincipalTag().enableInApiMetrics()
+        && config.realmIdTag().enableInApiMetrics()) {
       return ProductionReadinessCheck.of(
           Error.of(
-              "Metrics configuration includes user principal name in tags.",
+              "Metrics configuration includes both user principal name and realm id in tags this could cause performance implications.",
               "polaris.metrics.user-principal-tag.enable-in-api-metrics"));
+    } else {
+      LOGGER.warn(
+          "Metrics configuration includes user principal name in tags. "
+              + "This could have performance implications.");
     }
     return ProductionReadinessCheck.OK;
   }