dsmiley commented on a change in pull request #1921: URL: https://github.com/apache/lucene-solr/pull/1921#discussion_r496366133
########## File path: solr/solr-ref-guide/src/common-query-parameters.adoc ########## @@ -307,11 +307,13 @@ The `echoParams` parameter controls what information about request parameters is The `echoParams` parameter accepts the following values: -* `explicit`: This is the default value. Only parameters included in the actual request, plus the `_` parameter (which is a 64-bit numeric timestamp) will be added to the `params` section of the response header. +* `explicit`: Only parameters included in the actual request, plus the `_` parameter (which is a 64-bit numeric timestamp) will be added to the `params` section of the response header. Review comment: @gerlowskija this reminds me... (off topic of this PR), do you know where and why "_" parameter is added? If it's for query identification purposes then it's duplicitous with the "rid" you added. I'm having trouble searching for it. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org