clintropolis commented on a change in pull request #11676:
URL: https://github.com/apache/druid/pull/11676#discussion_r707896229
##########
File path:
services/src/main/java/org/apache/druid/server/AsyncQueryForwardingServlet.java
##########
@@ -219,6 +219,7 @@ protected void service(HttpServletRequest request,
HttpServletResponse response)
String connectionId = getAvaticaProtobufConnectionId(protobufRequest);
targetServer = hostFinder.findServerAvatica(connectionId);
request.setAttribute(AVATICA_QUERY_ATTRIBUTE, requestBytes);
+ LOG.debug("Forwarding protobuf JDBC connection [%s] to broker [%s]",
connectionId, targetServer);
Review comment:
Should there be an option to make these newly added debug logs be info
level? I've always found the router logs to be basically worthless unless stuff
is exploding, and letting it print stuff about what it is forwarding to where
without the overhead of the massive amount of debug level logs might be useful.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
