[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69270641 retest this please --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3939 --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69281768 The wiki location seems fine. Maybe others disagree. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69287974 Okie doke, thank you @andrewor14. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69277310 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25275/ Test PASSed. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69277301 [Test build #25275 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25275/consoleFull) for PR 3939 at commit [`66e0841`](https://github.com/apache/spark/commit/66e0841132331d0283ffdbd7a8e8203a67bd9d77). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69270905 [Test build #25275 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25275/consoleFull) for PR 3939 at commit [`66e0841`](https://github.com/apache/spark/commit/66e0841132331d0283ffdbd7a8e8203a67bd9d77). * This patch merges cleanly. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69281611 Ok LGTM I'm merging this into master thanks. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69257859 [Test build #25269 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25269/consoleFull) for PR 3939 at commit [`ceeab85`](https://github.com/apache/spark/commit/ceeab85619992cf3ae0183c6f57fb45bf118e940). * This patch merges cleanly. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22686463 --- Diff: ec2/spark_ec2.py --- @@ -706,9 +697,7 @@ def wait_for_cluster_state(conn, opts, cluster_instances, cluster_state): sys.stdout.flush() start_time = datetime.now() - num_attempts = 0 -conn = ec2.connect_to_region(opts.region) --- End diff -- It opens a connection to EC2. But since we are [already passed in a connection as `conn`](https://github.com/apache/spark/pull/3939#discussion-diff-22621704L693), there is no need to recreate this. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69258000 [Test build #25269 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25269/consoleFull) for PR 3939 at commit [`ceeab85`](https://github.com/apache/spark/commit/ceeab85619992cf3ae0183c6f57fb45bf118e940). * This patch **fails Python style tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69258006 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25269/ Test FAILed. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22686489 --- Diff: ec2/spark_ec2.py --- @@ -39,10 +39,24 @@ from optparse import OptionParser from sys import stderr +VALID_SPARK_VERSIONS = set([ +0.7.3, +0.8.0, +0.8.1, +0.9.0, +0.9.1, --- End diff -- Added. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69258425 @andrewor14 Yeah, I've tested launching: * valid Spark release * invalid Spark release * valid Spark hash * invalid Spark hash They all get to the build/deploy phase OK. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69258608 [Test build #25270 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25270/consoleFull) for PR 3939 at commit [`66e0841`](https://github.com/apache/spark/commit/66e0841132331d0283ffdbd7a8e8203a67bd9d77). * This patch merges cleanly. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22697350 --- Diff: ec2/spark_ec2.py --- @@ -706,9 +697,7 @@ def wait_for_cluster_state(conn, opts, cluster_instances, cluster_state): sys.stdout.flush() start_time = datetime.now() - num_attempts = 0 -conn = ec2.connect_to_region(opts.region) --- End diff -- I see. That does seem to be the case --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69261120 Btw @shivaram or @andrewor14, can you confirm that the wiki page I created (linked to in the PR body) is in the right place? --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on a diff in the pull request:
https://github.com/apache/spark/pull/3939#discussion_r22689661
--- Diff: ec2/spark_ec2.py ---
@@ -236,6 +252,26 @@ def get_or_make_group(conn, name, vpc_id):
return conn.create_security_group(name, Spark EC2 group, vpc_id)
+def get_validate_spark_version(version, repo):
+if . in version:
+version = version.replace(v, )
+if version not in VALID_SPARK_VERSIONS:
+print stderr, Don't know about Spark version:
{v}.format(v=version)
+sys.exit(1)
+return version
+else:
--- End diff --
FYI: I added validation of git hashes here.
---
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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69265837 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25270/ Test FAILed. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69265830 [Test build #25270 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25270/consoleFull) for PR 3939 at commit [`66e0841`](https://github.com/apache/spark/commit/66e0841132331d0283ffdbd7a8e8203a67bd9d77). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69266853 The failed test is a Kafka streaming test and is unrelated to this PR. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22670523 --- Diff: ec2/spark_ec2.py --- @@ -983,6 +969,12 @@ def real_main(): (opts, action, cluster_name) = parse_args() # Input parameter validation +if . in opts.spark_version: --- End diff -- Will do. I asked about getting rid of the `replace(v, )` so I wouldn't have to do that, but that didn't fly. :) --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22676551 --- Diff: ec2/spark_ec2.py --- @@ -39,10 +39,24 @@ from optparse import OptionParser from sys import stderr +VALID_SPARK_VERSIONS = set([ +0.7.3, +0.8.0, +0.8.1, +0.9.0, +0.9.1, --- End diff -- not your code, but where is `0.9.2`? --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69238256 LGTM from a quick glance. @nchammas have you had a chance to test this on a real cluster? --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22676840 --- Diff: ec2/spark_ec2.py --- @@ -983,6 +969,12 @@ def real_main(): (opts, action, cluster_name) = parse_args() # Input parameter validation +if . in opts.spark_version: --- End diff -- +1 on failing fast here --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22676711 --- Diff: ec2/spark_ec2.py --- @@ -706,9 +697,7 @@ def wait_for_cluster_state(conn, opts, cluster_instances, cluster_state): sys.stdout.flush() start_time = datetime.now() - num_attempts = 0 -conn = ec2.connect_to_region(opts.region) --- End diff -- what does this do? Why is it safe to remove? --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/3939#discussion_r22636258 --- Diff: ec2/spark_ec2.py --- @@ -983,6 +969,12 @@ def real_main(): (opts, action, cluster_name) = parse_args() # Input parameter validation +if . in opts.spark_version: --- End diff -- Minor comment: Since we do this spark_version.replace(v, ) in two places it might make sense to keep the old function but name it as `get_spark_version` (or `check_get_spark_version`). You can then call it once here and once while constructing the template variables --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69137984 I left a minor comment inline. LGTM otherwise. cc @JoshRosen --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69112089 [Test build #25176 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25176/consoleFull) for PR 3939 at commit [`fc0d5b9`](https://github.com/apache/spark/commit/fc0d5b954b605a1e73e5a0ffa99d21da). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69112093 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25176/ Test PASSed. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69114565 [Test build #25178 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25178/consoleFull) for PR 3939 at commit [`db4935d`](https://github.com/apache/spark/commit/db4935d8221d6af0e756c0452be5bf590c80f2db). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-5122] Remove Shark from spark-ec2
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3939#issuecomment-69114578 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25178/ Test PASSed. --- 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
