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

Zoltan Haindrich commented on HIVE-23110:
-----------------------------------------

I see that you've changed the the ticket's summary - the description is the 
place where it says: "Click to add description"

Wouldn't returning null as CommandProcessorResponse cause any trouble? I think 
if this method returns w/o an exception it means that it have finished 
correctly...
Shouldn't we lower the log priority to debug or trace here: 
https://github.com/apache/hive/blob/d2ad5b061706a1d3cd55e59c769ed4f2af01cdbe/service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java#L239
note that the current solution will change the state of sqlop to finished from 
canceled - not sure what that could cause (I guess nothing - but for an aborted 
query I think canceled might be better)

> Prevent NPE in ReExecDriver if the processing is aborted and thus the context 
> is null
> -------------------------------------------------------------------------------------
>
>                 Key: HIVE-23110
>                 URL: https://issues.apache.org/jira/browse/HIVE-23110
>             Project: Hive
>          Issue Type: Bug
>          Components: HiveServer2
>            Reporter: Miklos Gergely
>            Assignee: Miklos Gergely
>            Priority: Major
>         Attachments: HIVE-23110.01.patch
>
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to