[
https://issues.apache.org/jira/browse/TAJO-1282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14270477#comment-14270477
]
ASF GitHub Bot commented on TAJO-1282:
--------------------------------------
Github user jinossy commented on the pull request:
https://github.com/apache/tajo/pull/334#issuecomment-69287398
Could you remove {{isQueryMasterStopped}} in {{QueryInProgress.stop()}} ?
It seems unnecessary to wait.
> Cleanup the relationship of QueryInProgress and QueryJobManager
> ---------------------------------------------------------------
>
> Key: TAJO-1282
> URL: https://issues.apache.org/jira/browse/TAJO-1282
> Project: Tajo
> Issue Type: Improvement
> Components: tajo master
> Reporter: Hyunsik Choi
> Assignee: Hyunsik Choi
> Fix For: 0.10
>
> Attachments: TAJO-1282.patch, TAJO-1282_2.patch
>
>
> Each QueryInProgress instance maintains an individual event handler involving
> each thread. It complicates the relationship between QueryJobManager and
> QueryInprogress. The main objective of this issue is to remove each event
> handler from QueryInProgress and distinguishes their roles of both classes.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
