[
https://issues.apache.org/jira/browse/TAJO-1199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14224166#comment-14224166
]
ASF GitHub Bot commented on TAJO-1199:
--------------------------------------
Github user hyunsik commented on the pull request:
https://github.com/apache/tajo/pull/269#issuecomment-64321768
The patch looks good to me. I'd like to suggest two things.
* One is long option which is more descriptive. For example, it would be
great if the script provides ```--tar``` (```-t```), ```--conf``` (```-c```),
and ```--lib``` (```-l```) .
* The second is more modularization. Currently, multiple steps are a
sequence of routines. Could you modularize them into multiple pieces of
functions?
> EMR bootstrap script for Tajo
> -----------------------------
>
> Key: TAJO-1199
> URL: https://issues.apache.org/jira/browse/TAJO-1199
> Project: Tajo
> Issue Type: New Feature
> Components: tools
> Affects Versions: 0.9.1
> Reporter: YeonSu Han
> Assignee: YeonSu Han
> Labels: features
> Fix For: 0.9.1
>
>
> Hi guys,
> I'd like to contribute EMR bootstrap script for Tajo. With this script, you
> can easily launch a Tajo cluster on EMR.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
