You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/09/23 21:00:37 UTC

[GitHub] [druid] a2l007 commented on pull request #10426: more timeout handling in JsonParserIterator

a2l007 commented on pull request #10426:
URL: https://github.com/apache/druid/pull/10426#issuecomment-697969996


   This might not be within the scope of this PR, but do you think it would be useful to separate out timeout exceptions from `QueryInterruptedException`? We could have something like a `QueryTimeoutException` being thrown in case of timeouts and `QueryResource` can handle this separately by returning a 504 status instead of 500. I feel returning the 504 error code would be more useful for cluster operators to differentiate between query timeouts vs other types of failures.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org