[
https://issues.apache.org/jira/browse/TAJO-630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13917320#comment-13917320
]
Hudson commented on TAJO-630:
-----------------------------
SUCCESS: Integrated in Tajo-master-build #97 (See
[https://builds.apache.org/job/Tajo-master-build/97/])
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=15c4576ef680e05be877e9bb5b4370a03857c446)
* 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)