Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14805
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/14805
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64564/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14805
**[Test build #64564 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64564/consoleFull)**
for PR 14805 at commit
[`36c99f8`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14805
**[Test build #64564 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64564/consoleFull)**
for PR 14805 at commit
[`36c99f8`](https://github.com/apache/spark/commit/3
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14805
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 wishe
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14805
OK, it's looking good to me. I wouldn't mind a set of eyes more familiar
with Python to look at it. In particular just want to make sure we're headed
towards more standardization in all cases.
---
Github user silentsokolov commented on the issue:
https://github.com/apache/spark/pull/14805
I returned the spaces
---
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 Stibbons commented on the issue:
https://github.com/apache/spark/pull/14805
For information, autopep8 will do all the space changes automatically
---
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
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14805
Look good to me. I think it good to have the space arround kwargs
assignation removed to distinguish beween
```
a = 1
```
and
```
afunc(a=1)
```
which is a good habit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14805
@Stibbons since you're around, do you have any comments on these Python
style changes?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user silentsokolov commented on the issue:
https://github.com/apache/spark/pull/14805
I looked through all the python code examples, fixed typos and formatting
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14805
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64414/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14805
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/14805
**[Test build #64414 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64414/consoleFull)**
for PR 14805 at commit
[`92310d9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14805
**[Test build #64414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64414/consoleFull)**
for PR 14805 at commit
[`92310d9`](https://github.com/apache/spark/commit/9
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14805
OK, can you perhaps quickly search for other instances of the same in
Python code? it's worth a skim if you're up for it.
---
If your project is set up for it, you can reply to this email and have y
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14805
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 wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14805
Can one of the admins verify this patch?
---
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 feat
18 matches
Mail list logo
