Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22724
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97457/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
**[Test build #97457 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97457/testReport)**
for PR 22724 at commit
[`04a535d`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97457 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97457/testReport)**
for PR 22724 at commit
[`04a535d`](https://github.com/apache/spark/commit/04
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/4035/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
Merged build finished. Test FAILed.
---
-
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/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97451/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97451/testReport)**
for PR 22724 at commit
[`a2f0288`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22724
yea +1 on 3.0 only, this is kind of a developer API, advanced users may use
it.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22724
yes, you're right. So we don't need any migration note. But what if a user
uses `Literal` directly? He/she shouldn't, but we cannot exclude this is
happening. So I'd still target this for 3.0 only.
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22724
`Literal` is an internal API, and AFAIK end-users can't construct an
invalid `Literal` with public APIs. If they can, then it's a bug and we have a
problem...
---
--
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22724
yea, its ok to do so.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22724
LGTM, but this is a breaking change, so I'd suggest to do it only for 3.0
and add a note to the migration guide. WDYT?
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97451/testReport)**
for PR 22724 at commit
[`a2f0288`](https://github.com/apache/spark/commit/a2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
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/4031/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97443/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97443 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97443/testReport)**
for PR 22724 at commit
[`77f65f9`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97439/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
**[Test build #97439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97439/testReport)**
for PR 22724 at commit
[`60a7793`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
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/4027/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97443/testReport)**
for PR 22724 at commit
[`77f65f9`](https://github.com/apache/spark/commit/77
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97439/testReport)**
for PR 22724 at commit
[`60a7793`](https://github.com/apache/spark/commit/60
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
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/4024/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
**[Test build #97433 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97433/testReport)**
for PR 22724 at commit
[`1c042f3`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97433/
Test FAILed.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22724
I'm checking the reason of the test failures...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97432/testReport)**
for PR 22724 at commit
[`4df5abd`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97432/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
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/22724
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/4022/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22724
LGTM pending jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97433 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97433/testReport)**
for PR 22724 at commit
[`1c042f3`](https://github.com/apache/spark/commit/1c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97431 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97431/testReport)**
for PR 22724 at commit
[`56b4778`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97431/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
**[Test build #97432 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97432/testReport)**
for PR 22724 at commit
[`4df5abd`](https://github.com/apache/spark/commit/4d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/4021/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
**[Test build #97431 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97431/testReport)**
for PR 22724 at commit
[`56b4778`](https://github.com/apache/spark/commit/56
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
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/4020/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97427/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
**[Test build #97427 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97427/testReport)**
for PR 22724 at commit
[`a30e9ce`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
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/4017/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22724
**[Test build #97427 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97427/testReport)**
for PR 22724 at commit
[`a30e9ce`](https://github.com/apache/spark/commit/a3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97419/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
**[Test build #97419 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97419/testReport)**
for PR 22724 at commit
[`b00f4bb`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/4009/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
**[Test build #97419 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97419/testReport)**
for PR 22724 at commit
[`b00f4bb`](https://github.com/apache/spark/commit/b0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97384/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
Merged build finished. Test FAILed.
---
-
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/22724
**[Test build #97384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97384/testReport)**
for PR 22724 at commit
[`9115d26`](https://github.com/apache/spark/commit/9
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
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/3976/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22724
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/22724
**[Test build #97384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97384/testReport)**
for PR 22724 at commit
[`9115d26`](https://github.com/apache/spark/commit/91
69 matches
Mail list logo