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

Hudson commented on TAJO-630:
-----------------------------

SUCCESS: Integrated in Tajo-0.8.0-build #4 (See 
[https://builds.apache.org/job/Tajo-0.8.0-build/4/])
TAJO-630: QueryMasterTask never finished when Internal error occurs. (hyunsik: 
https://git-wip-us.apache.org/repos/asf/?p=incubator-tajo.git&a=commit&h=5178ac28bc58e394e8d5d879e7acebc1913bff3a)
* CHANGES.txt
* 
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/querymaster/QueryMasterTask.java


> QueryMasterTask never finished when Internal error occurs.
> ----------------------------------------------------------
>
>                 Key: TAJO-630
>                 URL: https://issues.apache.org/jira/browse/TAJO-630
>             Project: Tajo
>          Issue Type: Bug
>    Affects Versions: 0.8-incubating, 1.0-incubating
>            Reporter: hyoungjunkim
>            Assignee: Hyunsik Choi
>            Priority: Minor
>             Fix For: 0.8-incubating, 1.0-incubating
>
>         Attachments: TAJO-630.patch
>
>
> Query state is changed to QUERY_ERROR when QueryEventType.INTERNAL_ERROR 
> occurs, and QueryFinishEventHandler invoked. 
> However, there is no check logic for QUERY_ERROR. So stopQuery() never 
> called. 
> QueryMasterTask.java
> {code}
>   private class QueryFinishEventHandler implements 
> EventHandler<QueryMasterQueryCompletedEvent> {
>     @Override
>     public void handle(QueryMasterQueryCompletedEvent event) {
>       QueryId queryId = event.getQueryId();
>       LOG.info("Query completion notified from " + queryId);
>       while (!isTerminatedState(query.getState())) {
>         try {
>           synchronized (this) {
>             wait(10);
>           }
>         } catch (InterruptedException e) {
>           LOG.error(e);
>         }
>       }
>       LOG.info("Query final state: " + query.getState());
>       queryMasterContext.stopQuery(queryId);
>     }
>     private boolean isTerminatedState(QueryState state) {
>       return
>           state == QueryState.QUERY_SUCCEEDED ||
>           state == QueryState.QUERY_FAILED ||
>           state == QueryState.QUERY_KILLED;
>     }
>   }
> {code}



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to