yihua opened a new pull request, #9746:
URL: https://github.com/apache/hudi/pull/9746
### Change Logs
This PR moves log merging and memory-related configs to `hudi-common` module
so the common file group read can use them. Configs in `HoodieRealtimeConfig`
are removed if duplicate or merged to existing config if the functionality to
control is the same.
- Moved from `HoodieCompactionConfig` to `HoodieReaderConfig`:
`hoodie.compaction.lazy.block.read`(with alternative key
`compaction.lazy.block.read.enabled`), `hoodie.compaction.reverse.log.read`
- `HoodieMemoryConfig` is moved from `hudi-client-common` to `hudi-common`
module
- `HoodieRealtimeConfig` is removed.
- `compaction.memory.fraction` is merged with
`hoodie.memory.compaction.fraction`. Default value for MR is kept.
- `compaction.lazy.block.read.enabled` is merged with
`hoodie.compaction.lazy.block.read`.
- `hoodie.memory.dfs.buffer.max.size` already exists in
`HoodieMemoryConfig`. Default value for MR is kept.
- `hoodie.memory.spillable.map.path` already exists in
`HoodieMemoryConfig`.
- `hoodie.optimized.log.blocks.scan.enable` is moved to
`HoodieReaderConfig`.
### Impact
Code refactoring towards consistent logic in log merging and file group
reading.
### Risk level
none
### Documentation Update
N/A
### Contributor's checklist
- [ ] Read through [contributor's
guide](https://hudi.apache.org/contribute/how-to-contribute)
- [ ] Change Logs and Impact were stated clearly
- [ ] Adequate tests were added if applicable
- [ ] CI passed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
