[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-07 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-119237187 Merged into master. Thanks! --- 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-8788] [ML] Add Java unit test for PCA t...

2015-07-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7184 --- 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-8788] [ML] Add Java unit test for PCA t...

2015-07-07 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-119164340 OK with this one @jkbradley ? seems reasonable to me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118364851 [Test build #36503 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36503/consoleFull) for PR 7184 at commit

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118364368 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

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118370909 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-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7184#discussion_r33868622 --- Diff: mllib/src/test/java/org/apache/spark/ml/feature/JavaPCASuite.java --- @@ -0,0 +1,114 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118364340 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

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118370711 [Test build #36503 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36503/console) for PR 7184 at commit

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-117960097 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

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-117960071 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

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-117972762 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-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-117960241 [Test build #36376 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36376/consoleFull) for PR 7184 at commit

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-117972680 [Test build #36376 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36376/console) for PR 7184 at commit

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/7184 [SPARK-8788] [ML] Add Java unit test for PCA transformer Add Java unit test for PCA transformer You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/7184#discussion_r33765375 --- Diff: mllib/src/test/java/org/apache/spark/ml/feature/JavaPCASuite.java --- @@ -0,0 +1,114 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/7184#discussion_r33765367 --- Diff: mllib/src/test/java/org/apache/spark/ml/feature/JavaPCASuite.java --- @@ -0,0 +1,114 @@ +/* + * Licensed to the Apache Software Foundation