rangareddy commented on issue #16920: URL: https://github.com/apache/hudi/issues/16920#issuecomment-5391428983
This issue was reviewed as part of the JIRA-migrated backlog triage (HUDI-9200). **Findings: fixed and already closed - recording the evidence for the record.** Fixed by PR #19415, `refactor(reader): seal the shared merge path in KeyBasedFileGroupRecordBuffer`, merged 2026-07-31 as commit `29b42cbae382`. Verified on `master`, in `hudi-common/src/main/java/org/apache/hudi/common/table/read/buffer/KeyBasedFileGroupRecordBuffer.java`: ```java :113 public final void processNextDataRecord(BufferedRecord<T> record, Serializable recordKey) throws IOException :158 public final boolean isPartialMergingEnabled() ``` both with javadoc (`:105`, `:155`) explaining why they are sealed - which is exactly what this ticket asked for: preventing `PositionBasedFileGroupRecordBuffer` from overriding the default behaviour of the shared merge path. No further action needed here. One loose end: **JIRA HUDI-9200 is still Open/Unresolved**, so it is worth resolving to match this closure. -- 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]
