[
https://issues.apache.org/jira/browse/DRILL-4586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15453942#comment-15453942
]
ASF GitHub Bot commented on DRILL-4586:
---------------------------------------
Github user sudheeshkatkam commented on the issue:
https://github.com/apache/drill/pull/567
I am confused here. The patch enhances the catch clauses of thrown
exceptions and, in a way, "corrects" what was being reported to the user as a
SYSTEM (server) error to CLIENT error since these exceptions happened on the
client side (abd eventually a UserException needs to be reported).
I tried to extend UserException to ClientException, but no contextual
information can be added because of how the UserException.Builder is setup. I
will punt this.
> Create CLIENT ErrorType
> -----------------------
>
> Key: DRILL-4586
> URL: https://issues.apache.org/jira/browse/DRILL-4586
> Project: Apache Drill
> Issue Type: Improvement
> Reporter: Sudheesh Katkam
> Assignee: Parth Chandra
>
> To display client errors with nice messages, we use "system error". However
> system error which is not meant to be used when we want to display a proper
> error message. System errors are meant for unexpected errors that don't have
> a "nice" error message yet.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
