rangareddy commented on issue #17091: URL: https://github.com/apache/hudi/issues/17091#issuecomment-5391457925
This issue was reviewed as part of the JIRA-migrated backlog triage (HUDI-9596). **Findings: still open.** The body carries only JIRA boilerplate, so which APIs and what "clarity" means are unstated. One timing consideration worth recording: `BufferedRecordMerger` is the centre of the current merge design - `BufferedRecordMergerFactory` provides distinct `COMMIT_TIME_ORDERING` and `EVENT_TIME_ORDERING` mergers, each with partial-update variants, plus a `deltaMerge(DeleteRecord, BufferedRecord)` path - and RFC-97 (#17074, deprecating payload classes) is still in flight with its substantive PR open. So this API surface is likely to move again before it settles. Refactoring for clarity is usually best done after the shape stabilises, not during. Worth either sequencing this after RFC-97 lands or narrowing it to naming changes that are safe either way. Keeping this open. -- 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]
