Merged
Conversation
…with Tez(#3630) Co-authored-by: dylonyu <dylonyu@tencent.com>
Co-authored-by: Sivabalan Narayanan <n.siva.b@gmail.com>
…es from data table can trigger table services in metadata table (#3900)
…no commit data (#3928) Co-authored-by: yuzhaojing <yuzhaojing@bytedance.com>
…es. (#3873) * [HUDI-2634] Improved the metadata table bootstrap for very large tables. Following improvements are implemented: 1. Memory overhead reduction: - Existing code caches FileStatus for each file in memory. - Created a new class DirectoryInfo which is used to cache a director's file list with parts of the FileStatus (only filename and file len). This reduces the memory requirements. 2. Improved parallelism: - Existing code collects all the listing to the Driver and then creates HoodieRecord on the Driver. - This takes a long time for large tables (11million HoodieRecords to be created) - Created a new function in SparkRDDWriteClient specifically for bootstrap commit. In it, the HoodieRecord creation is parallelized across executors so it completes fast. 3. Fixed setting to limit the number of parallel listings: - Existing code had a bug wherein 1500 executors were hardcoded to perform listing. This leads to exception due to limit in the spark's result memory. - Corrected the use of the config. Result: Dataset has 1299 partitions and 12Million files. file listing time=1.5mins HoodieRecord creation time=13seconds deltacommit duration=2.6mins Co-authored-by: Sivabalan Narayanan <n.siva.b@gmail.com>
…y GenericRecord and Row (#3944)
…ithmeticException (#3955) - ExternalSpillableMap does the payload/value size estimation on the first put to determine when to spill over to disk map. The payload size re-estimation also happens after a minimum threshold of puts. This size re-estimation goes my the current in-memory map size for calculating average payload size and does attempts divide by zero operation when the map is size is empty. Avoiding the ArithmeticException during the payload size re-estimate by checking the map size upfront.
…nReadRollbackActionExecutor (#3978) - With rollback after first commit support added to metadata table, these test cases are safe to have metadata table turned on.
…ls.getRealtimeSplits (#3994)
Co-authored-by: 闫杜峰 <yandufeng@sinochem.com>
…ix and TestHoodieClientMultiWriter test fixes (#4384) - Made FileSystemBasedLockProviderTestClass thread safe and fixed the tryLock retry logic. - Made TestHoodieClientMultiWriter. testHoodieClientBasicMultiWriter deterministic in verifying the HoodieWriteConflictException.
[HUDI-3008] Fixing HoodieFileIndex partition column parsing for nested fields
…with empty checkpoint (#4334) * Adding ability to read entire data with HoodieIncrSource with empty checkpoint * Addressing comments
…o deserialize Avro binaries (#4353)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tips
What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)
Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Committer checklist
Has a corresponding JIRA in PR title & commit
Commit message is descriptive of the change
CI is green
Necessary doc changes done or have another open PR
For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.