Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
Hello. Sadly I cannot work on this we are in a middle of a big
restructuration at work.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Gentle follow up ping. I've got some bandwith next week.
---
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 ueshin commented on the issue:
https://github.com/apache/spark/pull/14830
Hi, are you still working on this?
---
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 a
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Sure, if you have a chance to rebase & check if any other changes are
needed that would be useful.
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
I guess a rebased will be welcomed, I can do it by tomorow if you want
---
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 projec
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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/14830
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75632/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #75632 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75632/testReport)**
for PR 14830 at commit
[`582c822`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #75632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75632/testReport)**
for PR 14830 at commit
[`582c822`](https://github.com/apache/spark/commit/58
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Jenkins retest 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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73445/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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/14830
**[Test build #73445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73445/testReport)**
for PR 14830 at commit
[`582c822`](https://github.com/apache/spark/commit/5
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Jenkins retest 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 wis
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
lets do a jenkins re-run just to make sure everything is up to date and
I'll try and get a final pass done soon. I think it would be good to improve
our examples to be closer to pep8 style for the s
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14830
Thanks @Stibbons. FWIW, I won't stay against but am just neutral. Let me
please defer to @holdenk and @srowen.
---
If your project is set up for it, you can reply to this email and have your
re
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
Hello. This is actually the execution of the pylint/autopep8 config
proposed in #14963. I can minimize a little bit more this PR by ignoring indeed
more rules.
---
If your project is set up for i
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14830
@Stibbons are there maybe some options in autopep8 to minimise the changes?
(just in case I believe we ignore some rules such as E402,E731,E241,W503 and
E226 in Spark).
---
If your project is
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14830
Thank you for cc'ing me @holdenk. Let me try to take a look within tomorrow
too at my best.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Great, thanks for updating this :) Would be good to see if @HyukjinKwon has
anything to say otherwise I'll do another pass through this tomorrow and
hopefully its really close :)
---
If your proje
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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/14830
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72862/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #72862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72862/testReport)**
for PR 14830 at commit
[`582c822`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 this feature
enabled and wis
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #72862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72862/testReport)**
for PR 14830 at commit
[`582c822`](https://github.com/apache/spark/commit/58
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #72861 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72861/testReport)**
for PR 14830 at commit
[`31cea6d`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72861/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #72861 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72861/testReport)**
for PR 14830 at commit
[`31cea6d`](https://github.com/apache/spark/commit/31
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 this feature
enabled and wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72859/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 this feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 feature
enabled and wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71079/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #71079 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71079/testReport)**
for PR 14830 at commit
[`78b66d8`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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/14830
**[Test build #71079 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71079/testReport)**
for PR 14830 at commit
[`78b66d8`](https://github.com/apache/spark/commit/78
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #71076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71076/testReport)**
for PR 14830 at commit
[`ff6aabf`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71076/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #71076 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71076/testReport)**
for PR 14830 at commit
[`ff6aabf`](https://github.com/apache/spark/commit/ff
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
Can we agree on merging this patch ?
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 have this fe
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14830
@holdenk no hurry, esp. given the holidays, just keeping it on the radar. I
can check back in at the end of the week
---
If your project is set up for it, you can reply to this email and have your
r
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Oh hey sorry, I'm on vacation right now so I'm a little slow on the PR
reviews, I'll try and take some time this week to take a look :)
---
If your project is set up for it, you can reply to this e
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14830
Another call for Python eyeballs -- looks like nice cleanup but it touches
117 example files
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14830
@davies or @holdenk do you have an opinion about the backslash-vs-parens
change? the rest looks like OK style improvements
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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/14830
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70478/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #70478 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70478/testReport)**
for PR 14830 at commit
[`2ca8eb7`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #70478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70478/testReport)**
for PR 14830 at commit
[`2ca8eb7`](https://github.com/apache/spark/commit/2c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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/14830
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70477/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #70477 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70477/testReport)**
for PR 14830 at commit
[`eb39a10`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #70477 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70477/testReport)**
for PR 14830 at commit
[`eb39a10`](https://github.com/apache/spark/commit/eb
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
Rebased, space issue will be fixed in another PR
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14830
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70474/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #70474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70474/consoleFull)**
for PR 14830 at commit
[`a4df13f`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14830
**[Test build #70474 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70474/consoleFull)**
for PR 14830 at commit
[`a4df13f`](https://github.com/apache/spark/commit/a
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
Rebased
---
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 f
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Great 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 have this feature
enabled and wishes so, or i
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14830
Sure !
---
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 fe
65 matches
Mail list logo