Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
Thanks all! I will create a followup to add alias OneHotEncoderEstimator
later.
---
-
To unsubscribe, e-mail:
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/23100
I went through the PR again, and it looks right to me. Merged into master.
Thanks!
---
-
To unsubscribe, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
Yes, this diff just reflect renaming OneHotEncoderEstimator to
OneHotEncoder. Besides that, this also changes related documents and example
codes.
---
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/23100
It's hard to track the huge diffs on renaming. I don't go though it
line-by-line. But if they're just renaming, the rest LGTM.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99367/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99367 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99367/testReport)**
for PR 23100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99366/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99366 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99366/testReport)**
for PR 23100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
Except for changing `Since` tag, is there any other comments?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99367 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99367/testReport)**
for PR 23100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99366/testReport)**
for PR 23100 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
@srowen Thanks for review. I think there is no R counterpart for
`OneHotEncoder`.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99168/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99168 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99168/testReport)**
for PR 23100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99168 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99168/testReport)**
for PR 23100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99161/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99161 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99161/testReport)**
for PR 23100 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
retest this please...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99152/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99152 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99152/testReport)**
for PR 23100 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99161 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99161/testReport)**
for PR 23100 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99154/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23100
**[Test build #99154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99154/testReport)**
for PR 23100 at commit
40 matches
Mail list logo