patsonluk commented on code in PR #1237:
URL: https://github.com/apache/solr/pull/1237#discussion_r1047482742
##########
solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java:
##########
@@ -326,6 +326,9 @@ protected ResponseBuilder newResponseBuilder(
@Override
public void handleRequestBody(SolrQueryRequest req, SolrQueryResponse rsp)
throws Exception {
+ if (log.isInfoEnabled()) {
+ log.info("Start Search Query: {}", req.getParamString());
+ }
Review Comment:
Thank you for the review! I do understand your concern of extra log info,
it's important for our use case as queries can sometimes be totally stuck hence
timeout wouldn't get printed.
I do agree that it's probably quite specific to our own scenarios (and we
might not really seeing issues like that anymore), perhaps changing it to debug
will be more suitable?
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
