Pinot: Add configuration validation to prevent setting both broker-url and proxy.enabled

Author: shreyas-dev

plugin/trino-pinot/src/main/java/io/trino/plugin/pinot/PinotConfig.java

@@ -270,4 +270,10 @@ public boolean allUrlSchemesEqual()
                 .distinct()
                 .count() == 1;
     }
+
+    @AssertTrue(message = "Invalid configuration: pinot.broker-url and pinot.proxy.enabled cannot both be set.")
+    public boolean isValidBrokerProxyConfiguration()
+    {
+        return !(brokerUrl.isPresent() && proxyEnabled);
+    }
 }

plugin/trino-pinot/src/main/java/io/trino/plugin/pinot/client/PinotClient.java

@@ -170,7 +170,7 @@ public PinotClient(
                 asyncReloading(CacheLoader.from(this::getAllTables), executor));
         this.controllerAuthenticationProvider = controllerAuthenticationProvider;
         this.brokerAuthenticationProvider = brokerAuthenticationProvider;
-        brokerHostAndPort = config.getBrokerUrl();
+        this.brokerHostAndPort = config.getBrokerUrl();
     }
 
     public static void addJsonBinders(JsonCodecBinder jsonCodecBinder)