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

Jihoon Son commented on TAJO-50:
--------------------------------

+1 
This patch looks good to me.
                
> Cleanup SubQuery
> ----------------
>
>                 Key: TAJO-50
>                 URL: https://issues.apache.org/jira/browse/TAJO-50
>             Project: Tajo
>          Issue Type: Improvement
>          Components: master
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>             Fix For: 0.2-incubating
>
>         Attachments: TAJO-50.patch
>
>
> This patch cleans up tajo.master.SubQuery. 
> This patch does as follows:
> * Add setStartTime() and setFinishTime() methods
> * Clean up SubQuery::cleanUp() and rename it to buildAndSetTableMeta()
> * Clean up InitAndRequestContainer::transition method
>  ** Divide codes into multiple methods named for their objectives
>  ** Make the control flow very simple
> * Add some methods to access member variables of SubQuery
> * Remove tajo.master.Priority

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to