[GitHub] spark pull request: [SPARK-14100][ML] Merging Estimator and Model:...
Github user jkbradley closed the pull request at: https://github.com/apache/spark/pull/11983 --- 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-14100][ML] Merging Estimator and Model:...
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-204570963 Closing. See JIRA for decision to cancel this proposal. --- 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-14100][ML] Merging Estimator and Model:...
Github user DeepSparkBot commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-204489694 @mengxr Please advise. --- 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-203608637 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54540/ 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-203608631 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 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-203608144 **[Test build #54540 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54540/consoleFull)** for PR 11983 at commit [`367d25c`](https://github.com/apache/spark/commit/367d25c3467320dcbfcad984d93120861ce36a16). * 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-203548664 **[Test build #54540 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54540/consoleFull)** for PR 11983 at commit [`367d25c`](https://github.com/apache/spark/commit/367d25c3467320dcbfcad984d93120861ce36a16). --- 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202769627 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 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202769632 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54413/ 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202769378 **[Test build #54413 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54413/consoleFull)** for PR 11983 at commit [`41dbb2c`](https://github.com/apache/spark/commit/41dbb2cfa0eaa3d767b2a425645b8227a901cb71). * This patch **fails PySpark 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202727216 **[Test build #54413 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54413/consoleFull)** for PR 11983 at commit [`41dbb2c`](https://github.com/apache/spark/commit/41dbb2cfa0eaa3d767b2a425645b8227a901cb71). --- 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-14100][ML] Merging Estimator and Model:...
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202726994 @mengxr This was actually easier when ```labels``` was stored as a Param; that made save/load handle it automatically. I just changed it to store labels as a private Param, and it worked locally. What do you think? --- 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202718460 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54408/ 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202718456 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 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202718037 **[Test build #54408 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54408/consoleFull)** for PR 11983 at commit [`1fb78f5`](https://github.com/apache/spark/commit/1fb78f5a8289c904c858e0eb4176b2263ade7f01). * 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202695914 **[Test build #54408 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54408/consoleFull)** for PR 11983 at commit [`1fb78f5`](https://github.com/apache/spark/commit/1fb78f5a8289c904c858e0eb4176b2263ade7f01). --- 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202166556 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54289/ 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202166555 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 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202166496 **[Test build #54289 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54289/consoleFull)** for PR 11983 at commit [`62fdb86`](https://github.com/apache/spark/commit/62fdb863b267445df321cdd5366a7af67c79084c). * 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202142461 **[Test build #54289 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54289/consoleFull)** for PR 11983 at commit [`62fdb86`](https://github.com/apache/spark/commit/62fdb863b267445df321cdd5366a7af67c79084c). --- 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202002406 **[Test build #2699 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2699/consoleFull)** for PR 11983 at commit [`df6a3da`](https://github.com/apache/spark/commit/df6a3dac26c0a6624dc4789e7bb780ebaff315d8). * This patch **fails Scala 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202002397 **[Test build #54280 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54280/consoleFull)** for PR 11983 at commit [`df6a3da`](https://github.com/apache/spark/commit/df6a3dac26c0a6624dc4789e7bb780ebaff315d8). * This patch **fails Scala 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202002399 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54280/ 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202002398 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 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202002051 **[Test build #2699 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2699/consoleFull)** for PR 11983 at commit [`df6a3da`](https://github.com/apache/spark/commit/df6a3dac26c0a6624dc4789e7bb780ebaff315d8). --- 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-202002015 **[Test build #54280 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54280/consoleFull)** for PR 11983 at commit [`df6a3da`](https://github.com/apache/spark/commit/df6a3dac26c0a6624dc4789e7bb780ebaff315d8). --- 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201970903 **[Test build #54271 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54271/consoleFull)** for PR 11983 at commit [`bf14a24`](https://github.com/apache/spark/commit/bf14a24f9ba91fdb7719c98027e5486ccbe79854). * This patch **fails MiMa 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201970906 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54271/ 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201970905 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 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201968589 **[Test build #54271 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54271/consoleFull)** for PR 11983 at commit [`bf14a24`](https://github.com/apache/spark/commit/bf14a24f9ba91fdb7719c98027e5486ccbe79854). --- 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201967263 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54270/ 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201967255 **[Test build #54270 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54270/consoleFull)** for PR 11983 at commit [`aa4b408`](https://github.com/apache/spark/commit/aa4b408a077bd3905693c8f6428eb682b5dc47b4). * This patch **fails to build**. * 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201967262 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 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201966705 **[Test build #54270 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54270/consoleFull)** for PR 11983 at commit [`aa4b408`](https://github.com/apache/spark/commit/aa4b408a077bd3905693c8f6428eb682b5dc47b4). --- 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201964442 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 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-14100][ML] Merging Estimator and Model:...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201964443 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54269/ 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201964439 **[Test build #54269 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54269/consoleFull)** for PR 11983 at commit [`bc06166`](https://github.com/apache/spark/commit/bc0616605091f77d6c9621fc55f5d3561ba5a05d). * This patch **fails Scala style tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `abstract class Model[M <: Model[M]] extends Transformer ` * `abstract class MutableEstimator[T <: MutableEstimator[T]] extends Transformer ` * `class MutableEstimator(Transformer):` * `class StringIndexer(JavaMutableEstimator, HasInputCol, HasOutputCol, HasHandleInvalid,` * `class JavaMutableEstimator(MutableEstimator, JavaTransformer):` --- 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-14100][ML] Merging Estimator and Model:...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201964119 **[Test build #54269 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54269/consoleFull)** for PR 11983 at commit [`bc06166`](https://github.com/apache/spark/commit/bc0616605091f77d6c9621fc55f5d3561ba5a05d). --- 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-14100][ML] Merging Estimator and Model:...
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/11983#issuecomment-201963929 CC: @mengxr Here's the prototype --- 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-14100][ML] Merging Estimator and Model:...
GitHub user jkbradley opened a pull request: https://github.com/apache/spark/pull/11983 [SPARK-14100][ML] Merging Estimator and Model: prototype for StringIndexer ## What changes were proposed in this pull request? This is a *prototype*. It will be used to decide whether or not to proceed with [https://issues.apache.org/jira/browse/SPARK-14100]. Main changes * Created new abstraction MutableEstimator which will eventually replace Estimator for Spark 2.0. * MutableEstimator inherits from Transformer, and it contains method ```fit()```. * It does not contain fit() methods taking ParamMaps. The expected behavior of such methods becomes more ambiguous since it is unclear if they modify the current instance. * Merged StringIndexer and StringIndexerModel, where the merged abstraction now inherits from MutableEstimator. * Did the same for the Python API. Also added JavaMutableEstimator for Python wrappers. Other required changes * Modified Pipeline to handle MutableEstimator. Other proposed changes * Deprecated transform() methods in Transformer taking Param settings. * Added ```copy()``` without arguments to PipelineStage since this will be a more common operation after the merge. * Added more ```set()``` methods to Params to facilitate setting Param values, now that fit() and transform() methods taking ParamMaps are going to be removed. ## How was this patch tested? Existing unit tests. Note that the required changes were minimal. More changes to meta-algorithms such as CrossValidator may be needed as we merge other Estimator-Model pairs. You can merge this pull request into a Git repository by running: $ git pull https://github.com/jkbradley/spark thunterdb-14100 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/11983.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 #11983 commit 0585e3fbc6dc7316ae2d8dd425648c9f6b45e041 Author: Timothy HunterDate: 2016-03-15T21:54:20Z passing tests commit 6aa439bb78aea37476e7a12209a9f902a7be9871 Author: Timothy Hunter Date: 2016-03-15T21:56:36Z cleanups commit 317df204c049a08c3e230c4d3ca61ea6f122c864 Author: Timothy Hunter Date: 2016-03-23T21:01:59Z wokr commit bc0616605091f77d6c9621fc55f5d3561ba5a05d Author: Joseph K. Bradley Date: 2016-03-26T22:24:28Z Made StringIndexer extend MutableEstimator in Python and Scala. --- 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