Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/16536
Thanks a bunch @holdenk
---
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
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16536
Going to go ahead and merge. Still need to sort out the JIRA permissions so
will take a bit for me to get that updated for you.
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/7/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #7 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/7/testReport)**
for PR 16536 at commit
[`923b88d`](https://github.com/apache/spark/commit/9
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16536
Great, I'll wait for jenkins then :)
---
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
enable
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/16536
@holdenk 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 feature
enabled and wishes so, or
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #7 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/7/testReport)**
for PR 16536 at commit
[`923b88d`](https://github.com/apache/spark/commit/92
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16536
@zero323 that sounds like a good improvement.
---
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 featu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72219 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72219/testReport)**
for PR 16536 at commit
[`9332da3`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72219/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72218/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #72218 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72218/testReport)**
for PR 16536 at commit
[`cb496f3`](https://github.com/apache/spark/commit/c
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/16536
@holdenk I have one more suggestion. Shouldn't we replace
```python
def _create_judf(self):
from pyspark.sql import SparkSession
sc = SparkContext.getOrCr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72219 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72219/testReport)**
for PR 16536 at commit
[`9332da3`](https://github.com/apache/spark/commit/93
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72218 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72218/testReport)**
for PR 16536 at commit
[`cb496f3`](https://github.com/apache/spark/commit/cb
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16536
The changes look good to me, I'll take a quick pass at the formatting to
make sure - but otherwise I'll try and merge this tomorrow :)
---
If your project is set up for it, you can reply to this em
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #72173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72173/testReport)**
for PR 16536 at commit
[`1a8280d`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72173/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72172/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #72172 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72172/testReport)**
for PR 16536 at commit
[`489ef54`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72170/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72170 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72170/testReport)**
for PR 16536 at commit
[`ee953a9`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72173/testReport)**
for PR 16536 at commit
[`1a8280d`](https://github.com/apache/spark/commit/1a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72172 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72172/testReport)**
for PR 16536 at commit
[`489ef54`](https://github.com/apache/spark/commit/48
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72170 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72170/testReport)**
for PR 16536 at commit
[`ee953a9`](https://github.com/apache/spark/commit/ee
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72039/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #72039 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72039/testReport)**
for PR 16536 at commit
[`29ffc57`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72039 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72039/testReport)**
for PR 16536 at commit
[`29ffc57`](https://github.com/apache/spark/commit/29
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72034/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #72034 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72034/testReport)**
for PR 16536 at commit
[`a5567b2`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72034 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72034/testReport)**
for PR 16536 at commit
[`a5567b2`](https://github.com/apache/spark/commit/a5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72021/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72021 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72021/testReport)**
for PR 16536 at commit
[`16245e4`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #72021 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72021/testReport)**
for PR 16536 at commit
[`16245e4`](https://github.com/apache/spark/commit/16
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/16536
+1
Looks good to me.
---
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 wi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71688/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #71688 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71688/testReport)**
for PR 16536 at commit
[`abb3726`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #71688 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71688/testReport)**
for PR 16536 at commit
[`abb3726`](https://github.com/apache/spark/commit/ab
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71679/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #71679 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71679/testReport)**
for PR 16536 at commit
[`57735b2`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #71679 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71679/testReport)**
for PR 16536 at commit
[`57735b2`](https://github.com/apache/spark/commit/57
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71350/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #71350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71350/testReport)**
for PR 16536 at commit
[`b124bb4`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #71351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71351/testReport)**
for PR 16536 at commit
[`641ec1d`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71351/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16536
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/16536
**[Test build #71351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71351/testReport)**
for PR 16536 at commit
[`641ec1d`](https://github.com/apache/spark/commit/64
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16536
**[Test build #71350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71350/testReport)**
for PR 16536 at commit
[`b124bb4`](https://github.com/apache/spark/commit/b1
60 matches
Mail list logo
