You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/09/01 01:17:20 UTC

[jira] [Commented] (DRILL-4586) Create CLIENT ErrorType

    [ 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)