hudi icon indicating copy to clipboard operation
hudi copied to clipboard

[HUDI-9526][DRAFT] Use HoodieFileGroupReader throughout the CDC flow

Open the-other-tim-brown opened this issue 5 months ago • 1 comments

Change Logs

  • Update spark reader to use HoodieFileGroupReader
  • Add logic for easily checking if a row represents a delete operation to avoid converting to and from avro multiple times in the CDC spark reader
  • TODO: compaction support

Impact

Describe any public API or user-facing feature change or any performance impact.

Risk level (write none, low medium or high below)

If medium or high, explain what verification was done to mitigate the risks.

Documentation Update

Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

  • The config description must be updated if new configs are added or the default value of the configs are changed
  • Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the ticket number here and follow the instruction to make changes to the website.

Contributor's checklist

  • [ ] Read through contributor's guide
  • [ ] Change Logs and Impact were stated clearly
  • [ ] Adequate tests were added if applicable
  • [ ] CI passed

the-other-tim-brown avatar Jun 16 '25 15:06 the-other-tim-brown

@danny0405 Can you please list out your remaining concerns as a checklist..

vinothchandar avatar Jul 08 '25 15:07 vinothchandar

As discussed, I can shepherd this PR once @yihua chimes in.

vinothchandar avatar Jul 09 '25 14:07 vinothchandar

@danny0405 Can you please list out your remaining concerns as a checklist..

Left some comments.

danny0405 avatar Jul 10 '25 03:07 danny0405

@jonvex you plan to review this?

vinothchandar avatar Jul 17 '25 22:07 vinothchandar

CI report:

  • 15a27a8e0312a55fc7e536c4bf91d15edd3294a0 UNKNOWN
  • 93425d152b8586d862d0f5fa4a58f69b12628986 Azure: SUCCESS
Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

hudi-bot avatar Jul 19 '25 13:07 hudi-bot