-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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-7576] Improve efficiency of getRelativePartitionPath, reduce computation of partitionPath in AbstractTableFileSystemView #11001
Conversation
… partitionPath in AbstractTableFileSystemView
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.
+1, I love this fix.
@the-other-tim-brown Can you fix the Azure CI failure? |
@danny0405 error is:
I'll put up a separate minor pr to make the ordering deterministic for small file handling |
@danny0405 can you take another look when you get a chance? I have updated a few spots in the code |
…mputation of partitionPath in AbstractTableFileSystemView (#11001)
…mputation of partitionPath in AbstractTableFileSystemView (#11001)
Change Logs
getRelativePartitionPath
by reducing the number of operations on the path object that are required to get the final resultImpact
Reduces overhead of building FSViews with large numbers of files
Risk level (write none, low medium or high below)
None
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".
ticket number here and follow the instruction to make
changes to the website.
Contributor's checklist