Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
LGTM, merging to master.
---
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/14959
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/14959
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66758/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #66758 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66758/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #66758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66758/consoleFull)**
for PR 14959 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
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 wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66700/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
**[Test build #66700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66700/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #66700 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66700/consoleFull)**
for PR 14959 at commit
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/14959
Thanks for the updates @zjffdu, just 2 minor comments otherwise LGTM!
---
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 holdenk commented on the issue:
https://github.com/apache/spark/pull/14959
Awesome, thanks for updating. I'm at PyData this weekend so will be a bit
slow on my end.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/14959
@vanzin @holdenk @BryanCutler PR is updated, please help review.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66588/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #66588 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66588/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #66588 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66588/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
**[Test build #66587 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66587/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66587/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #66587 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66587/consoleFull)**
for PR 14959 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14959
I'd also love to see this updated so we can finish the review and make it
easier for people to use PySpark this way :)
---
If your project is set up for it, you can reply to this email and have
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
@zjffdu could you update the PR? 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
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/14959
I added a few comments @zjffdu. I also tested this PR out and looks good
---
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
LGTM. Let's see if others have any comments.
---
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 zjffdu commented on the issue:
https://github.com/apache/spark/pull/14959
I just fix the unti test, @vanzin @davies @holdenk Mind to help review it
? Thanks
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65699/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65699 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65699/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65699 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65699/consoleFull)**
for PR 14959 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
Are these tests flaky or is the failure related to this change? (Other PRs
seem to be passing, so probably the latter?)
---
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/14959
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/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65649/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65649 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65649/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65649 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65649/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65641/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65641 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65641/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65641 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65641/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65639/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65639 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65639/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
**[Test build #65639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65639/consoleFull)**
for PR 14959 at commit
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/14959
```
The internal SparkConf of the context will not be the same instance as conf.
```
This is the existing implementation that python is different from scala.
But I think it is correct. I
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
It means that if you do this:
val conf = new SparkConf()
val sc = new SparkContext(conf)
The internal SparkConf of the context will not be the same instance as
`conf`.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
**[Test build #65551 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65551/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65551/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #65551 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65551/consoleFull)**
for PR 14959 at commit
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/14959
@vanzin Thanks for your reviews. I just update the PR, but don't get your
following statement mean. Can you explain it ? Thanks
```
Especially since the Scala SparkContext clones the original
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
Also someone else more familiar with pyspark (I know Holden has already
looked), maybe @davies, should take a look.
---
If your project is set up for it, you can reply to this email and have your
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14959
The conf code looks kinda nasty with the checks for whether a JVM has been
set or not... I guess part of it is mandatory because otherwise this wouldn't
work, but in particular, I'm not so sure the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64937/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
**[Test build #64937 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64937/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #64937 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64937/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #64935 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64935/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64935/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #64935 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64935/consoleFull)**
for PR 14959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64933/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14959
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/14959
**[Test build #64933 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64933/consoleFull)**
for PR 14959 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14959
**[Test build #64933 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64933/consoleFull)**
for PR 14959 at commit
64 matches
Mail list logo