Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21682
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/21682
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92781/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92781 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92781/testReport)**
for PR 21682 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92781 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92781/testReport)**
for PR 21682 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
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/797/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21682
ping @wangyum
---
-
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/21682
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92589/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92589 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92589/testReport)**
for PR 21682 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
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/654/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92589 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92589/testReport)**
for PR 21682 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21682
+1
I agree with some of the minor refactoring suggestions, but overall this
looks correct to me.
---
-
To unsubscribe,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21682
Wow .. so it was 4 years ago .. okay.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21682
It is a regression that was introduced in Spark 1.2. Almost 4 years ago.
https://issues.apache.org/jira/browse/SPARK-4453
Thus, I think no need to document it. The pushdown can be
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21682
re: https://github.com/apache/spark/pull/21682#issuecomment-401676889 - I
couldn't recall related things about it. Thanks @gatorsmile. So, is it a
regression? shall we update the migration
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21682
@gatorsmile aha, thanks.
---
-
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/21682
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92519/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92519 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92519/testReport)**
for PR 21682 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21682
@maropu We accidentally dropped the pushdown of these data types when we
refactored the file formats. You can check the change history and find the PR.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21682
cc @gengliangwang
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21682
@HyukjinKwon btw, why we don't support pushdown for these types? Any
historical reason?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92519 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92519/testReport)**
for PR 21682 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
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/612/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21682
retest this please
---
-
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/21682
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92503/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92503 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92503/testReport)**
for PR 21682 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
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/603/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21682
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21682
**[Test build #92503 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92503/testReport)**
for PR 21682 at commit
35 matches
Mail list logo