[ 
https://issues.apache.org/jira/browse/HIVE-5230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13778703#comment-13778703
 ] 

Brock Noland commented on HIVE-5230:
------------------------------------

This looks good to me, meaning +1. One question, is possible that in 
checkExecutionStatus, while we are calling getBackgrounException() that another 
thread would call setBackgrounException()?
                
> Better error reporting by async threads in HiveServer2
> ------------------------------------------------------
>
>                 Key: HIVE-5230
>                 URL: https://issues.apache.org/jira/browse/HIVE-5230
>             Project: Hive
>          Issue Type: Improvement
>          Components: HiveServer2
>            Reporter: Vaibhav Gumashta
>            Assignee: Prasad Mujumdar
>             Fix For: 0.13.0
>
>         Attachments: HIVE-5230.1.patch, HIVE-5230.1.patch
>
>
> [HIVE-4617|https://issues.apache.org/jira/browse/HIVE-4617] provides support 
> for async execution in HS2. When a background thread gets an error, currently 
> the client can only poll for the operation state and also the error with its 
> stacktrace is logged. However, it will be useful to provide a richer error 
> response like thrift API does with TStatus (which is constructed while 
> building a Thrift response object). 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to