[ https://issues.apache.org/jira/browse/WHIRR-415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13147161#comment-13147161 ]
Tom White commented on WHIRR-415: --------------------------------- The tarball.url overload was a hack to get ideas on how to do this better! Alex, your patch is much better. However, the decision to use environment variables vs. properties is an implementation detail, which I'm not sure is something the user cares about. (I also worry about the ability to set arbitrary environment variables.) E.g. in this case the setting should be "whirr.hadoop.repo=cdh3u1" or similar. Perhaps we can just have another property for that? > Let users specify the CDH release (cdh3u1, cdh3u2) > -------------------------------------------------- > > Key: WHIRR-415 > URL: https://issues.apache.org/jira/browse/WHIRR-415 > Project: Whirr > Issue Type: Improvement > Components: service/cdh > Reporter: Andrei Savu > Priority: Critical > Fix For: 0.7.0 > > Attachments: WHIRR-415-env.patch, WHIRR-415.patch > > > Allow explicit selection of the CDH release you want to use. The current > behaviour is to use the latest one. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira