[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-31 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-136521155 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-8472] [ML] [PySpark] Python API for DCT

2015-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8485 --- 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-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135959233 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-8472] [ML] [PySpark] Python API for DCT

2015-08-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135953111 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-8472] [ML] [PySpark] Python API for DCT

2015-08-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135953058 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-8472] [ML] [PySpark] Python API for DCT

2015-08-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135705308 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-8472] [ML] [PySpark] Python API for DCT

2015-08-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135705261 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-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135718765 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-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135436880 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-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135436933 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-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/8485 [SPARK-8472] [ML] [PySpark] Python API for DCT Add Python API for ml.feature.DCT. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135446096 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-8472] [ML] [PySpark] Python API for DCT

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

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8485#discussion_r38116030 --- Diff: python/pyspark/ml/feature.py --- @@ -167,6 +167,65 @@ def getSplits(self): @inherit_doc +class DCT(JavaTransformer, HasInputCol,

[GitHub] spark pull request: [SPARK-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8485#issuecomment-135485931 LGTM except minor inline comments. --- 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-8472] [ML] [PySpark] Python API for DCT

2015-08-27 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8485#discussion_r38116027 --- Diff: python/pyspark/ml/feature.py --- @@ -167,6 +167,65 @@ def getSplits(self): @inherit_doc +class DCT(JavaTransformer, HasInputCol,