[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/108 --- 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 enabl

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37217443 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13097/ --- If your project

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37217440 Merged build finished. --- 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

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37210259 Merged build started. --- 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 t

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37210258 Merged build triggered. --- 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 hav

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37207969 Jenkins, test 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 t

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37158466 One or more automated tests failed Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13089/ --- If your p

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37158463 Merged build finished. --- 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

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37156418 Merged build triggered. --- 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 hav

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37156420 Merged build started. --- 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 t

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread mateiz
Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37156275 Cool, looks good then. --- 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 fe

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37154826 Hey @mateiz updated the docs. --- 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 hav

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/108#discussion_r10417447 --- Diff: docs/monitoring.md --- @@ -48,11 +48,22 @@ Each instance can report to zero or more _sinks_. Sinks are contained in the * `ConsoleSink`:

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread mateiz
Github user mateiz commented on a diff in the pull request: https://github.com/apache/spark/pull/108#discussion_r10417262 --- Diff: docs/monitoring.md --- @@ -48,11 +48,22 @@ Each instance can report to zero or more _sinks_. Sinks are contained in the * `ConsoleSink`: Lo

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37137320 @aarondav the spark-ec2 scripts set up a ganglia cluster separately... they actually don't use the `GangliaSink` in Spark at present. If change spark ec2 to use that then

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37136294 Will this require the spark-ec2 scripts to pull in this dependency as well? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-09 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37135373 @mateiz if you could take a glance at this that would be great. This has to get merged into 0.9.1 so I want to make sure a few people look at it. --- If your project is

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37118026 Merged build finished. --- 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

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37118027 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13080/ --- If your project

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37117125 Merged build triggered. --- 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 hav

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37117126 Merged build started. --- 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 t

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37117121 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13079/ --- If your project

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37117120 Merged build finished. --- 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

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37116234 Merged build started. --- 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 t

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/108#issuecomment-37116233 Merged build triggered. --- 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 hav

[GitHub] spark pull request: SPARK-1167: Remove metrics-ganglia from defaul...

2014-03-08 Thread pwendell
GitHub user pwendell opened a pull request: https://github.com/apache/spark/pull/108 SPARK-1167: Remove metrics-ganglia from default build due to LGPL issues... This patch removes Ganglia integration from the default build. It allows users willing to link against LGPL code to use