This is an automated email from the ASF dual-hosted git repository.
siddteotia pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 56927a193a Disable logging request header by default. (#9192)
56927a193a is described below
commit 56927a193a31b42e542aec75390a4a0bdfac774a
Author: Vivek Iyer Vaidyanathan <vviveki...@gmail.com>
AuthorDate: Wed Aug 10 13:10:19 2022 -0700
Disable logging request header by default. (#9192)
---
.../src/main/java/org/apache/pinot/spi/utils/CommonConstants.java | 4 +++-
1 file changed, 3 insertions(+), 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
index 47866461ec..c24702b0dc 100644
--- 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 class CommonConstants {
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";
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
