[GitHub] spark pull request: [SPARK-5668] Display region in spark_ec2.py ge...

2015-02-10 Thread MiguelPeralvo
Github user MiguelPeralvo commented on the pull request: https://github.com/apache/spark/pull/4457#issuecomment-73728792 The MiMa fail may be flaky. It passed in my computer. Jenkins, retest this please. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-5668] Display region in spark_ec2.py ge...

2015-02-09 Thread MiguelPeralvo
Github user MiguelPeralvo commented on the pull request: https://github.com/apache/spark/pull/4457#issuecomment-73536251 @nchammas, Regarding the help text, I'd say that it applies to the destroy, login, reboot-slaves, get-master, stop and start options, not only launch

[GitHub] spark pull request: [SPARK-5668] Display region in spark_ec2.py ge...

2015-02-09 Thread MiguelPeralvo
Github user MiguelPeralvo commented on a diff in the pull request: https://github.com/apache/spark/pull/4457#discussion_r24331636 --- Diff: ec2/spark_ec2.py --- @@ -607,14 +608,17 @@ def get_existing_cluster(conn, opts, cluster_name, die_on_error=True): elif

[GitHub] spark pull request: [SPARK-5667] Remove version from spark-ec2 exa...

2015-02-08 Thread MiguelPeralvo
Github user MiguelPeralvo commented on the pull request: https://github.com/apache/spark/pull/4300#issuecomment-73376066 Thank you @nchammas, I've done as you suggested (created JIRA [SPARK-5667] and amended title). --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-5668] Display region in spark_ec2.py ge...

2015-02-08 Thread MiguelPeralvo
GitHub user MiguelPeralvo opened a pull request: https://github.com/apache/spark/pull/4457 [SPARK-5668] Display region in spark_ec2.py get_existing_cluster() Show the region for the different messages displayed by get_existing_cluster(): The search, found and error messages. You

[GitHub] spark pull request: Update ec2-scripts.md

2015-02-02 Thread MiguelPeralvo
Github user MiguelPeralvo commented on the pull request: https://github.com/apache/spark/pull/4300#issuecomment-72542008 I think that's even a better option. I'm happy to propose it. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: Update ec2-scripts.md

2015-02-02 Thread MiguelPeralvo
Github user MiguelPeralvo commented on the pull request: https://github.com/apache/spark/pull/4300#issuecomment-72543263 I've amended the commit. --- 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

[GitHub] spark pull request: Update ec2-scripts.md

2015-02-01 Thread MiguelPeralvo
GitHub user MiguelPeralvo opened a pull request: https://github.com/apache/spark/pull/4300 Update ec2-scripts.md Change spark-version from 1.1.0 to 1.2.0 in the example for spark-ec2/Launch Cluster. You can merge this pull request into a Git repository by running: $ git pull