[GitHub] spark pull request: [SPARK-4291][Build] Rename network module proj...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62180406 Ok, thanks @shivaram. I renamed it Spark Project Networking. What do others think about this name? --- 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-4291][Build] Rename network module proj...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62182350 [Test build #23057 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23057/consoleFull) for PR 3148 at commit [`eac839b`](https://github.com/apache/spark/commit/eac839b0c8524ae778b09c23b7296a1c75e51297). * 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-4291][Build] Rename network module proj...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62191364 [Test build #23057 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23057/consoleFull) for PR 3148 at commit [`eac839b`](https://github.com/apache/spark/commit/eac839b0c8524ae778b09c23b7296a1c75e51297). * 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-4291][Build] Rename network module proj...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62191373 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/23057/ 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-4291][Build] Rename network module proj...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62191626 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-4291][Build] Rename network module proj...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62192179 [Test build #23058 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23058/consoleFull) for PR 3148 at commit [`eac839b`](https://github.com/apache/spark/commit/eac839b0c8524ae778b09c23b7296a1c75e51297). * 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-4291][Build] Rename network module proj...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62204471 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/23058/ 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-4291][Build] Rename network module proj...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62204464 [Test build #23058 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23058/consoleFull) for PR 3148 at commit [`eac839b`](https://github.com/apache/spark/commit/eac839b0c8524ae778b09c23b7296a1c75e51297). * 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-4291][Build] Rename network module proj...
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62204703 SGTM! --- 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-4291][Build] Rename network module proj...
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62248744 LGTM - I'm pulling this in. --- 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-4291][Build] Rename network module proj...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3148 --- 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-4291][Build] Rename network module proj...
GitHub user andrewor14 opened a pull request: https://github.com/apache/spark/pull/3148 [SPARK-4291][Build] Rename network module project names The names of the recently introduced network modules are inconsistent with those of the other modules in the project: ``` [INFO] Reactor Build Order: [INFO] [INFO] Spark Project Parent POM [INFO] Spark Project Common Network Code [INFO] Spark Project Shuffle Streaming Service Code [INFO] Spark Project Core [INFO] Spark Project Bagel [INFO] Spark Project GraphX [INFO] Spark Project Streaming [INFO] Spark Project Catalyst [INFO] Spark Project SQL [INFO] Spark Project ML Library [INFO] Spark Project Tools [INFO] Spark Project Hive [INFO] Spark Project REPL [INFO] Spark Project YARN Parent POM [INFO] Spark Project YARN Stable API [INFO] Spark Project Assembly [INFO] Spark Project External Twitter [INFO] Spark Project External Kafka [INFO] Spark Project External Flume Sink [INFO] Spark Project External Flume [INFO] Spark Project External ZeroMQ [INFO] Spark Project External MQTT [INFO] Spark Project Examples [INFO] Spark Project Yarn Shuffle Service Code ``` We should just drop the Code suffix since it doesn't sacrifice any meaning. You can merge this pull request into a Git repository by running: $ git pull https://github.com/andrewor14/spark build-drop-code Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/3148.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #3148 commit d01ad47d8321bf8f9e56c3a4361632ddb09ac7b0 Author: Andrew Or and...@databricks.com Date: 2014-11-07T05:17:56Z Rename network module project names --- 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-4291][Build] Rename network module proj...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62097115 @pwendell Are these names 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-4291][Build] Rename network module proj...
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/3148#discussion_r19995127 --- Diff: network/common/pom.xml --- @@ -29,7 +29,7 @@ groupIdorg.apache.spark/groupId artifactIdspark-network-common_2.10/artifactId packagingjar/packaging - nameSpark Project Common Network Code/name + nameSpark Project Network/name --- End diff -- Networking or Networking Library instead of Network ? --- 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-4291][Build] Rename network module proj...
Github user sryza commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62097234 +1. Was about to suggest such a change. --- 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-4291][Build] Rename network module proj...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62097287 [Test build #23038 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23038/consoleFull) for PR 3148 at commit [`d01ad47`](https://github.com/apache/spark/commit/d01ad47d8321bf8f9e56c3a4361632ddb09ac7b0). * 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-4291][Build] Rename network module proj...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62106012 [Test build #23038 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23038/consoleFull) for PR 3148 at commit [`d01ad47`](https://github.com/apache/spark/commit/d01ad47d8321bf8f9e56c3a4361632ddb09ac7b0). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class StandaloneWorkerShuffleService(sparkConf: SparkConf, securityManager: SecurityManager)` * `public class RetryingBlockFetcher ` --- 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-4291][Build] Rename network module proj...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3148#issuecomment-62106016 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/23038/ 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
