Disable logging request header by default. (#9192)

apache · Aug 10, 2022 · 56927a1 · 56927a1
1 parent 6ed17d2
commit 56927a1
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java b/pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java
@@ -256,7 +256,9 @@ public static class Broker {
 
     public static final String CONFIG_OF_BROKER_REQUEST_CLIENT_IP_LOGGING =
         "pinot.broker.request.client.ip.logging";
-    public static final boolean DEFAULT_BROKER_REQUEST_CLIENT_IP_LOGGING = true;
+
+    // TODO: Support populating clientIp for GrpcRequestIdentity.
+    public static final boolean DEFAULT_BROKER_REQUEST_CLIENT_IP_LOGGING = false;
 
     public static class Request {
       public static final String SQL = "sql";