Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8102
---
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 enab
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-139083436
LGTM. Merging with master. Thank you!
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-138812922
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-138812824
[Test build #42195 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42195/console)
for PR 8102 at commit
[`67bf664`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-138812923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-138810764
[Test build #42195 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42195/consoleFull)
for PR 8102 at commit
[`67bf664`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-138810344
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-138810363
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132860927
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132860925
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132860849
[Test build #41298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41298/console)
for PR 8102 at commit
[`879474d`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132857608
[Test build #41298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41298/consoleFull)
for PR 8102 at commit
[`879474d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132855718
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132855676
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 h
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132854062
Jenkins, test 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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132560183
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 n
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132560185
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132560134
**[Test build #41230 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41230/console)**
for PR 8102 at commit
[`5d2f75b`](https://github
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132507685
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132507687
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132507461
[Test build #41232 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41232/console)
for PR 8102 at commit
[`879474d`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132502400
[Test build #41232 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41232/consoleFull)
for PR 8102 at commit
[`879474d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132500987
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132500890
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132498799
[Test build #41230 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41230/consoleFull)
for PR 8102 at commit
[`5d2f75b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132498096
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132498127
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 f
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132345003
One more comment: need to add VectorSlicer to list ```__all__``` at top of
file
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/8102#discussion_r37350810
--- Diff: python/pyspark/ml/feature.py ---
@@ -950,6 +950,92 @@ class VectorIndexerModel(JavaModel):
@inherit_doc
+class VectorSlicer(Ja
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/8102#discussion_r37350798
--- Diff: python/pyspark/ml/feature.py ---
@@ -950,6 +950,92 @@ class VectorIndexerModel(JavaModel):
@inherit_doc
+class VectorSlicer(Ja
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/8102#discussion_r37350812
--- Diff: python/pyspark/ml/feature.py ---
@@ -950,6 +950,92 @@ class VectorIndexerModel(JavaModel):
@inherit_doc
+class VectorSlicer(Ja
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/8102#discussion_r37350805
--- Diff: python/pyspark/ml/feature.py ---
@@ -950,6 +950,92 @@ class VectorIndexerModel(JavaModel):
@inherit_doc
+class VectorSlicer(Ja
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-132332280
reviewing 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 this featur
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-130872240
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-130872159
[Test build #40826 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40826/console)
for PR 8102 at commit
[`54bffab`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-130868979
[Test build #40826 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40826/consoleFull)
for PR 8102 at commit
[`54bffab`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-130868523
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-130868536
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 have
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8102#issuecomment-130868338
test 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 this fea
39 matches
Mail list logo
