Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22545
SGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22545
I'm okay with merging this only into master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22545
Anyway, merged to master.
Let me take #22552 out of branch-2.4 for now but please feel free to get
this and that into branch-2.4 (without checking along with me) if you guys feel
strong
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96663/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96663 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96663/testReport)**
for PR 22545 at commit
[`2ff180e`](https://github.com/apache/spark/commit/2
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22545
I think the session support is kind of partially implemented in Python
side, and not being very well tested. There are some inconsistency between
Python and Scala side (for instance see
https:/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22545
The scala side change is already in 2.3.0. If we are ok with the behavior
inconsistency between python and scala, it's fine to merge it to master only
(and revert #22552 from 2.4 as well).
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3514/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96663 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96663/testReport)**
for PR 22545 at commit
[`2ff180e`](https://github.com/apache/spark/commit/2f
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22545
@cloud-fan, do we target this 2.4? Looks it might break an existing app, in
particular, when a Python shell creates a session and another shell (like
Zeppelin) or another session depends on a co
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96655/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96655/testReport)**
for PR 22545 at commit
[`d8c60cb`](https://github.com/apache/spark/commit/d
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22545
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3508/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96655/testReport)**
for PR 22545 at commit
[`d8c60cb`](https://github.com/apache/spark/commit/d8
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22545
@dongjoon-hyun I'm sorry that I haven't shared the context, but I talked
with @cloud-fan about
https://github.com/apache/spark/pull/22545#discussion_r220414608 off-line, and
he wanted to separate t
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22545
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96553/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96553 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96553/testReport)**
for PR 22545 at commit
[`ac0243a`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96553 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96553/testReport)**
for PR 22545 at commit
[`ac0243a`](https://github.com/apache/spark/commit/ac
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3435/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96550 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96550/testReport)**
for PR 22545 at commit
[`52584d9`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96550/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3434/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22545
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22545
**[Test build #96550 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96550/testReport)**
for PR 22545 at commit
[`52584d9`](https://github.com/apache/spark/commit/52
33 matches
Mail list logo