Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1899#discussion_r17812732
--- Diff: ec2/spark_ec2.py ---
@@ -440,14 +449,29 @@ def launch_cluster(conn, opts, cluster_name):
print "Launched master in %s, regid = %s" % (zone,
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1899#discussion_r17812734
--- Diff: ec2/spark_ec2.py ---
@@ -440,14 +449,29 @@ def launch_cluster(conn, opts, cluster_name):
print "Launched master in %s, regid = %s" % (zone,
Github user douglaz commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-54001048
Opened an issue related to this PR:
https://issues.apache.org/jira/browse/SPARK-3180
---
If your project is set up for it, you can reply to this email and have your
repl
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1899
---
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 enab
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-52694532
Alright, great. I'm going to merge this into `master` and `branch-1.1`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-52686462
LGTM
---
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 a
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/1899#discussion_r16437659
--- Diff: ec2/spark_ec2.py ---
@@ -440,14 +449,29 @@ def launch_cluster(conn, opts, cluster_name):
print "Launched master in %s, regid = %s" % (z
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-52685205
@shivaram @pwendell This looks good to me. Does one of you want to take a
final look + signoff?
---
If your project is set up for it, you can reply to this email and
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-52580052
[QA tests have
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18799/consoleFull)
for PR 1899 at commit
[`c80d5c3`](https://github.com/a
Github user vidaha commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-52576663
Okay, I made the retry a try catch, and edited the title again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1899#issuecomment-52576660
[QA tests have
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18799/consoleFull)
for PR 1899 at commit
[`c80d5c3`](https://github.com/ap
11 matches
Mail list logo
