Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19110
LGTM
Merging with master
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81653/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81653 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81653/testReport)**
for PR 19110 at commit
[`c24d4e2`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81653 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81653/testReport)**
for PR 19110 at commit
[`c24d4e2`](https://github.com/apache/spark/commit/c2
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19110
Other than that 1 item, this looks ready
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19110
Thanks @MLnick @BryanCutler . Would you mind helping review another similar
PR #19122 ? We need some other features but blocking on that PR. Thanks!
---
--
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/19110
LGTM!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81456/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81456 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81456/testReport)**
for PR 19110 at commit
[`7a1d404`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81456 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81456/testReport)**
for PR 19110 at commit
[`7a1d404`](https://github.com/apache/spark/commit/7a
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19110
@MLnick Conflict resolved. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional command
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19110
FYI I went ahead and merged #16774 - the doc on the shared param trait is a
little more detailed there which I slightly prefer. @WeichenXu123 you will just
need to resolve the small merge conflict th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81438/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81438 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81438/testReport)**
for PR 19110 at commit
[`edcf85c`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81438 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81438/testReport)**
for PR 19110 at commit
[`edcf85c`](https://github.com/apache/spark/commit/ed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81436/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81436 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81436/testReport)**
for PR 19110 at commit
[`7d0849e`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81436 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81436/testReport)**
for PR 19110 at commit
[`7d0849e`](https://github.com/apache/spark/commit/7d
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19110
LGTM
Pinging on https://github.com/apache/spark/pull/16774 first to coordinate
merging
---
-
To unsubscribe, e-mail: revie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81381/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81381/testReport)**
for PR 19110 at commit
[`fc6fd5e`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81381/testReport)**
for PR 19110 at commit
[`fc6fd5e`](https://github.com/apache/spark/commit/fc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81352/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81352 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81352/testReport)**
for PR 19110 at commit
[`24f4499`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81352/testReport)**
for PR 19110 at commit
[`24f4499`](https://github.com/apache/spark/commit/24
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19110
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 have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19110
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81350/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81350/testReport)**
for PR 19110 at commit
[`24f4499`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19110
**[Test build #81350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81350/testReport)**
for PR 19110 at commit
[`24f4499`](https://github.com/apache/spark/commit/24
36 matches
Mail list logo