[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-16 Thread freeman-lab
Github user freeman-lab commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52398080 @mengxr this looks great, will be simpler than writing custom loaders, especially for streaming where there are so many possible input sources. --- If your project

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-16 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52407340 @freeman-lab Thanks for reviewing! I've merged this into master and branch-1.1. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/1952 --- 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

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52365382 Jenkins, 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

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52365764 QA tests have started for PR 1952. This patch DID NOT merge cleanly! brView progress:

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52366795 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18629/consoleFull) for PR 1952 at commit

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52372263 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18629/consoleFull) for PR 1952 at commit

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52372303 QA results for PR 1952:br- This patch FAILED unit tests.brbrFor more information see test

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52376838 Jenkins, 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

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52376968 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18645/consoleFull) for PR 1952 at commit

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52378542 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18645/consoleFull) for PR 1952 at commit

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-14 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/1952 [SPARK-3048][MLLIB] add LabeledPoint.parse and remove loadStreamingLabeledPoints Move `parse()` from `LabeledPointParser` to `LabeledPoint` and make it public. This breaks binary compatibility only

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52253251 QA tests have started for PR 1952. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18570/consoleFull ---

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52256207 QA results for PR 1952:br- This patch FAILED unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52258006 QA tests have started for PR 1952. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18580/consoleFull ---

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52261434 QA results for PR 1952:br- This patch FAILED unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: [SPARK-3048][MLLIB] add LabeledPoint.parse and...

2014-08-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1952#issuecomment-52273898 QA tests have started for PR 1952. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18593/consoleFull ---