deepakpanda93 commented on issue #14672: URL: https://github.com/apache/hudi/issues/14672#issuecomment-4893279054
This issue was reviewed as part of the JIRA-migrated backlog triage. Findings: This delegation refactor was implemented by #11597 ([HUDI-7692] Extract metadata record type to MetadataPartitionType enum, merged 2024-08-06, shipped in 1.0.0). HoodieMetadataPayload no longer carries the monolithic per-type merge logic — its preCombine now delegates via `MetadataPartitionType.get(type).combineMetadataPayloads(previousRecord, this)`, and each partition type (FILES, COLUMN_STATS, BLOOM_FILTERS, RECORD_INDEX, SECONDARY_INDEX, ...) implements its own `combineMetadataPayloads` override. This removed ~315 lines of inline `switch(type)` code from the payload class, making it a delegator exactly as proposed here. If you're still hitting this on a recent version, please reopen with details. Closing as fixed. -- 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]
