-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-17636][SQL][WIP] Parquet predicate pushdown in nested fields #22535
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Test build #96506 has started for PR 22535 at commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR will be a good performance improvement for Spark 2.5.0.
|
retest this please. |
|
Test build #96528 has finished for PR 22535 at commit
|
|
Hi, @dbtsai . Could you fix the UT failure? [info] ParquetFilterSuite:
...
[info] - SPARK-12218 Converting conjunctions into Parquet filter predicates *** FAILED *** (19 milliseconds)
[info] Expected None, but got Some(lt(a, 10)) (ParquetFilterSuite.scala:802) |
|
I'm breaking this PRs into three smaller PR. I'll fix the tests in those smaller PRs. Thanks. |
|
Test build #99673 has finished for PR 22535 at commit
|
c95706f to
49f19a9
Compare
|
Test build #101203 has finished for PR 22535 at commit
|
49f19a9 to
eb400ed
Compare
|
Test build #101204 has finished for PR 22535 at commit
|
|
@dbtsai are you still working on this? |
|
Test build #115067 has finished for PR 22535 at commit
|
|
Going to close this in favor of new PR, #27728 |
What changes were proposed in this pull request?
Support Parquet predicate pushdown in nested fields
How was this patch tested?
Existing tests and new tests are added.