-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[HUDI-7351] Implement partition pushdown for glue #10604
[HUDI-7351] Implement partition pushdown for glue #10604
Conversation
@danny0405 you were involved in the previous PR |
There are no UT, can you ensure the functionality by offline e2e tests? |
I e2e tested locally. We will land this patch next week in production so let me confirm it's all right then. |
BTW @danny0405 I am working on a POC for IT test with moto to improve the reliability of this module #10614 |
421f87a
to
7b94f04
Compare
7b94f04
to
c63161e
Compare
@danny0405 Introduced IT tests for hudi-aws with dockerized moto here |
Change Logs
This is a follow up of #10572
While the mentioned PR fixed the runtime error, it did not implement the logic for pushdown which was silently no returning partitions.
The current PR does so by:
Impact
Using this feature has proven faster metastore sync in case of large number of partitions, from couple of minutes to seconds.
Risk level (write none, low medium or high below)
Integration testing in aws environment has been done
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change
ticket number here and follow the instruction to make
changes to the website.
Contributor's checklist