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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
65 matches
Mail list logo