[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38982438 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +937,63 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-138707231 That should be it. 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38982448 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +937,63 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-138737289 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-138737304 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-138743470 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-138786877 Merging with 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7976 --- 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136828857 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136828901 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136831122 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136831124 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136840867 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136840864 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136827809 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136827786 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136832134 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136832132 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136838421 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136838388 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136600743 There's still a "." on L952. I agree my wording is not the clearest, do you have any thoughts about improving it? In any case, I think this issue is quite

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136605069 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136605130 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136613126 Yes, sorry I was trying to point out that it currently doesn't have one and needs one added. I like the doc change! --- If your project is set up for

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136603278 Shouldn't L952 have a . on it? --- 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136607001 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136607005 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136866370 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136862791 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-9654][ML][PYSPARK] Add IndexToString to...

2015-09-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136862767 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136532811 @holdenk do you mind resolving merge conflicts? --- 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136538064 @feynmanliang done :) --- 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136538146 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136538121 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136540968 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378014 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378017 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378157 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378187 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378222 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378240 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378256 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378231 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38378274 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136545801 LGTM, some really tiny nits --- 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136582578 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136582730 @feynmanliang I'm not sure I agree with the changes to the param docs your asking for. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136582566 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38384756 --- Diff: python/pyspark/ml/feature.py --- @@ -931,6 +936,64 @@ class StringIndexerModel(JavaModel): """ Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-136586434 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-135900353 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-28 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r38248302 --- Diff: python/pyspark/ml/feature.py --- @@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-135897666 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-135897649 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-24 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37726435 --- Diff: python/pyspark/ml/feature.py --- @@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-19 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37489806 --- Diff: python/pyspark/ml/feature.py --- @@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-19 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37489984 --- Diff: python/pyspark/ml/feature.py --- @@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-19 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37490235 --- Diff: python/pyspark/ml/feature.py --- @@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-19 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37490167 --- Diff: python/pyspark/ml/feature.py --- @@ -817,6 +822,64 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132384812 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 have

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132384848 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 have this

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132385864 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132385792 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132390284 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132387897 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132388482 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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132387202 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-9654][ML][PYSPARK] Add IndexToString to...

2015-08-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132387183 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-9654][ML][PYSPARK] Add IndexToString to...

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

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252846 --- Diff: python/pyspark/ml/feature.py --- @@ -760,6 +765,60 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252849 --- Diff: python/pyspark/ml/feature.py --- @@ -760,6 +765,60 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252835 --- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala --- @@ -122,6 +122,11 @@ class StringIndexerModel ( map }

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252845 --- Diff: python/pyspark/ml/feature.py --- @@ -760,6 +765,60 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252847 --- Diff: python/pyspark/ml/feature.py --- @@ -760,6 +765,60 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252837 --- Diff: python/pyspark/ml/feature.py --- @@ -760,6 +765,60 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/7976#issuecomment-132003360 Thanks for updating! I just made a new pass over it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-9654][ML][PYSPARK] Add IndexToString to...

2015-08-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/7976#discussion_r37252841 --- Diff: python/pyspark/ml/feature.py --- @@ -760,6 +765,60 @@ class StringIndexerModel(JavaModel): Model fitted by StringIndexer.

  1   2   >