Github user nchammas commented on the pull request:
https://github.com/apache/spark/pull/1120#issuecomment-54754216
Since this has been tested (both by hand and by Jenkins), I think the only
thing that remains is perhaps to add a note in
[ec2-scripts.md](https://github.com/apache/spark/blob/master/docs/ec2-scripts.md)
about this feature, as well as an additional note about how to override it
with different credentials if so desired. After that, I think this can be
merged in.
@JoshRosen Does that sound good to you?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
