rangareddy commented on issue #15057:
URL: https://github.com/apache/hudi/issues/15057#issuecomment-5068043232

   This issue was reviewed as part of the JIRA-migrated backlog triage.
   
   Findings: The linked PR #5087 ([HUDI-3614] Replace List with HoodieData in 
HoodieFlink/JavaTable and commit executors) was closed unmerged on 2023-10-27, 
and per the JIRA it was put on hold due to a perf concern from the refactoring. 
On current master the requested rework has not landed: FlinkDeleteHelper 
(hudi-client/hudi-flink-client/.../commit/FlinkDeleteHelper.java) and 
JavaDeleteHelper 
(hudi-client/hudi-java-client/.../commit/JavaDeleteHelper.java) still exist as 
separate classes operating on List<HoodieKey> with their own deduplicateKeys 
implementations; they were never merged into HoodieDeleteHelper and the API was 
not switched to HoodieData. This remains incomplete refactoring/dedup work.
   
   Keeping open. Note: the status:pr-available label is stale, since the only 
linked PR is closed-unmerged.


-- 
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]

Reply via email to