[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8290 --- 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] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132734888 Merging with master and branch-1.5 --- 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: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132720200 [Test build #41259 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41259/console) for PR 8290 at commit [`314b43e`](https://github.

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132720563 Merged build finished. 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132720566 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132697677 [Test build #41259 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41259/consoleFull) for PR 8290 at commit [`314b43e`](https://gith

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132696372 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132696347 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 h

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132647109 @feynmanliang Could you change the title to `ml.evaluation.Evaluator`? --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-19 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8290#discussion_r37424665 --- Diff: mllib/src/main/scala/org/apache/spark/ml/evaluation/Evaluator.scala --- @@ -46,5 +46,12 @@ abstract class Evaluator extends Params { */

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132454119 Jenkins test this please On Tue, Aug 18, 2015, 10:24 PM UCB AMPLab wrote: > Merged build finished. Test FAILed. > > — > Reply to this

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132450545 Merged build finished. 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132450546 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132450153 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132450142 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 h

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132407740 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132407735 Merged build finished. 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132407705 [Test build #41180 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41180/console) for PR 8290 at commit [`027eb3a`](https://github.

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132402689 LGTM pending tests --- 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 f

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132402676 [Test build #41180 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41180/consoleFull) for PR 8290 at commit [`027eb3a`](https://gith

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132401586 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132401573 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 h

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132401404 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 ha

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132399530 Merged build finished. 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132399538 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132394639 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 h

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132394650 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132391539 Merged build finished. 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132391542 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132391513 [Test build #41173 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41173/console) for PR 8290 at commit [`7b00d2e`](https://github.

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132387334 [Test build #41173 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41173/consoleFull) for PR 8290 at commit [`7b00d2e`](https://gith

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132385863 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

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8290#issuecomment-132385784 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 h

[GitHub] spark pull request: [SPARK-10097] Adds `shouldMaximize` flag to `m...

2015-08-18 Thread feynmanliang
GitHub user feynmanliang opened a pull request: https://github.com/apache/spark/pull/8290 [SPARK-10097] Adds `shouldMaximize` flag to `ml.tuning.Evaluator` Previously, users of evaluator (`CrossValidator` and `TrainValidationSplit`) would only maximize the metric in evaluator, leadi