[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-29 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-203134673 Sure Sent from my iPhone > On Mar 29, 2016, at 14:59, jkbradley wrote: > > Sure, I'll go ahead and review though.

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-29 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-203129484 Sure, I'll go ahead and review though. Btw, can you add the "[ML]" tag to the PR title? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-29 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-203127557 Not happen on me. Maybe you can hold off first, I can check it later today because I am off my PC now, then ping you if it passes. --- If your project is set up for

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-29 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-203085347 This says it does not conflict with master, but when I rebase on master, it has conflicts. Does this happen to you? --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-28 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-202726190 @jkbradley Ready to review. --- 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

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-202712339 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

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-202712341 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-202712245 **[Test build #54411 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54411/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-202702573 **[Test build #54411 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54411/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-28 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-202647482 Refactoring this one now. --- 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-11892] Model export/import for spark.ml...

2016-03-23 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-200555922 It's OK I think. They are not related. I'll rebase that one now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-200525026 **[Test build #2671 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2671/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-23 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-200509769 I'll take a look now --- 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-11892] Model export/import for spark.ml...

2016-03-23 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-200510661 @yinxusen Actually, should I wait to review this until https://github.com/apache/spark/pull/9971 gets merged? Which should come first? --- If your project is set up

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-03-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-200509787 **[Test build #2671 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2671/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-01-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-170488432 **[Test build #49131 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49131/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-01-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-170501604 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

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-01-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-170501607 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-01-11 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-170487788 Updated for 2.0 cc @jkbradley --- 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

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2016-01-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-170501450 **[Test build #49131 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49131/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2015-11-24 Thread yinxusen
GitHub user yinxusen opened a pull request: https://github.com/apache/spark/pull/9934 [SPARK-11892] Model export/import for spark.ml: OneVsRest It's for v1.7. https://issues.apache.org/jira/browse/SPARK-11892 You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2015-11-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-159284173 **[Test build #46604 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46604/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2015-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-159300684 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

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2015-11-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-159300365 **[Test build #46604 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46604/consoleFull)** for PR 9934 at commit

[GitHub] spark pull request: [SPARK-11892] Model export/import for spark.ml...

2015-11-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9934#issuecomment-159300691 Test PASSed. Refer to this link for build results (access rights to CI server needed):