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

ASF GitHub Bot commented on TAJO-751:
-------------------------------------

Github user blrunner commented on a diff in the pull request:

    https://github.com/apache/tajo/pull/459#discussion_r28495611
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java ---
    @@ -623,6 +623,12 @@ public BoolProto killQuery(RpcController controller, 
QueryIdRequest request) thr
           try {
             context.getSessionManager().touch(request.getSessionId().getId());
             QueryId queryId = new QueryId(request.getQueryId());
    +
    +        QueryInProgress progress = 
context.getQueryJobManager().getQueryInProgress(queryId);
    +        if (progress == null || progress.isFinishState() || 
progress.isFinishing()) {
    --- End diff --
    
    Do we need to check the kill wait status surely?


> JDBC driver should support cancel() method.
> -------------------------------------------
>
>                 Key: TAJO-751
>                 URL: https://issues.apache.org/jira/browse/TAJO-751
>             Project: Tajo
>          Issue Type: Improvement
>            Reporter: Hyoungjun Kim
>            Assignee: Navis
>            Priority: Minor
>
> Some OLAP or ETL tool call JDBC's cancel() function when a query is too late. 
> So Tajo's JDBC should support cancel() function in Statement or 
> PreparedStatement class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to