jihoonson commented on a change in pull request #10554:
URL: https://github.com/apache/druid/pull/10554#discussion_r521764393
##########
File path: docs/querying/querying.md
##########
@@ -117,9 +117,12 @@ If a query fails, you will get an HTTP 500 response
containing a JSON object wit
"host" : "druid1.example.com:8083"
}
```
+The HTTP response code returned depends on the type of query failure. For
timed out queries, an HTTP 504 response code is returned.
If a query request fails due to being limited by the [query scheduler laning
configuration](../configuration/index.md#broker), an HTTP 429 response with the
same JSON object schema as an error response, but with `errorMessage` of the
form: "Total query capacity exceeded" or "Query capacity exceeded for lane
'low'".
+For every other type of query failures, an HTTP 500 response code is returned.
Review comment:
The other missing response codes here are 401 and 403 which can be
returned for auth failures. Would you please add them as well?
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
