soumitra-st commented on code in PR #11136:
URL: https://github.com/apache/pinot/pull/11136#discussion_r1268613693
##########
pinot-broker/src/main/java/org/apache/pinot/broker/api/resources/PinotClientRequest.java:
##########
@@ -121,6 +121,8 @@ public void processSqlQueryGet(@ApiParam(value = "Query",
required = true) @Quer
}
BrokerResponse brokerResponse = executeSqlQuery(requestJson,
makeHttpIdentity(requestContext), true);
asyncResponse.resume(brokerResponse.toJsonString());
+ } catch (WebApplicationException wae) {
Review Comment:
Added log. There is a Meter metric
REQUEST_DROPPED_DUE_TO_ACCESS_ERROR("requestsDropped") which is tracking at
several places, but let me file a task to review and add at the missing places,
such as request filter.
--
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
