[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r940544395 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -658,9 +661,34 @@ private boolean isFilterAlwaysTrue(PinotQuery pi

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r933641715 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -658,9 +661,34 @@ private boolean isFilterAlwaysTrue(PinotQuery pi

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r933641715 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -658,9 +661,34 @@ private boolean isFilterAlwaysTrue(PinotQuery pi

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r93132 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -658,9 +661,34 @@ private boolean isFilterAlwaysTrue(PinotQuery pi

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r918462453 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -658,9 +661,32 @@ private boolean isFilterAlwaysTrue(PinotQuery pi

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r906395996 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -453,8 +455,13 @@ private BrokerResponseNative handleRequest(long

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r903158186 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -682,7 +697,8 @@ private void logBrokerResponse(long requestId, St

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r903158186 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -682,7 +697,8 @@ private void logBrokerResponse(long requestId, St

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r903158186 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -682,7 +697,8 @@ private void logBrokerResponse(long requestId, St

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r903157376 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -453,8 +455,13 @@ private BrokerResponseNative handleRequest(long

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r899442872 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -682,7 +685,9 @@ private void logBrokerResponse(long requestId, St

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r899419574 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -682,7 +685,9 @@ private void logBrokerResponse(long requestId, St

[GitHub] [pinot] abhs50 commented on a diff in pull request #8907: Log Request header for Broker

abhs50 commented on code in PR #8907: URL: https://github.com/apache/pinot/pull/8907#discussion_r899419574 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -682,7 +685,9 @@ private void logBrokerResponse(long requestId, St