gortiz commented on code in PR #12982:
URL: https://github.com/apache/pinot/pull/12982#discussion_r1574536612
##########
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java:
##########
@@ -309,7 +309,7 @@ protected BrokerResponse handleRequest(long requestId,
String query, @Nullable S
JsonNode request, @Nullable RequesterIdentity requesterIdentity,
RequestContext requestContext,
@Nullable HttpHeaders httpHeaders)
throws Exception {
- LOGGER.debug("SQL query for request {}: {}", requestId, query);
+ LOGGER.info("SQL query for request {}: {}", requestId, query);
Review Comment:
This would just double the lines, given we already emit a info log for each
query in `QueryLogger.log`, which is executed at the end of the request.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
