Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79036 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79036/testReport)**
for PR 18501 at commit
[`8517fe8`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
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/18501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79036/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Hi, @cloud-fan .
Could you review this 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 thi
Github user liguohua-bigdata commented on the issue:
https://github.com/apache/spark/pull/18501
ha
---
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 th
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79049 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79049/testReport)**
for PR 18501 at commit
[`b22369b`](https://github.com/apache/spark/commit/b2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79049 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79049/testReport)**
for PR 18501 at commit
[`b22369b`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79049/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Hi, @cloud-fan .
According to the comment in #18172, I removed that line because
`mergeSparkConf` will merge the configurations later. If I missed something,
please let me know. Thanks.
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
@cloud-fan and @gatorsmile .
In this PR, I'll revert to the first commit until your commit will be
merged.
For #18172 , I'm not sure the reason why it's reverted. But, as @cloud-fan
's
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79072 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79072/testReport)**
for PR 18501 at commit
[`8a1a64f`](https://github.com/apache/spark/commit/8a
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18501
My commit was reverted because my assumption was wrong. Some configurations
have to be set before creating `SparkContext`, so we can't just create
`SparkContext` and then set confs.
So th
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Thank you, @cloud-fan . I'll try like that.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79072 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79072/testReport)**
for PR 18501 at commit
[`8a1a64f`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79072/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79080 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79080/testReport)**
for PR 18501 at commit
[`8a1a64f`](https://github.com/apache/spark/commit/8a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79080 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79080/testReport)**
for PR 18501 at commit
[`8a1a64f`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
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/18501
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79080/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79127/testReport)**
for PR 18501 at commit
[`137f252`](https://github.com/apache/spark/commit/13
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Hi, @cloud-fan and @gatorsmile .
I'm back to this PR.
Although this introduces a new concept, this could be a solution in the
current relation between `SparkContext` and `SparkSession`
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18501
LGTM except one comment
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79127/testReport)**
for PR 18501 at commit
[`137f252`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
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/18501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79127/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Thank you so much, @cloud-fan . I updated it.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79155 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79155/testReport)**
for PR 18501 at commit
[`c9686f7`](https://github.com/apache/spark/commit/c9
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18501
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 project does not have this feature
enabled and wishes so, or if the fe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18501
**[Test build #79155 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79155/testReport)**
for PR 18501 at commit
[`c9686f7`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79155/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18501
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Thank you again, @gatorsmile .
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18501
Thanks! Merging to master/2.2
---
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 an
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18501
Thank you for review and merging, @gatorsmile and @cloud-fan .
---
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 d
37 matches
Mail list logo
