Skip to content

refactor: Add required setter methods for Flink-CDC#14150

Merged
danny0405 merged 1 commit into
apache:masterfrom
voonhous:flink-cdc-preqrequisite
Oct 27, 2025
Merged

refactor: Add required setter methods for Flink-CDC#14150
danny0405 merged 1 commit into
apache:masterfrom
voonhous:flink-cdc-preqrequisite

Conversation

@voonhous
Copy link
Copy Markdown
Member

Describe the issue this Pull Request addresses

Adding required (pre-requisites) for Hudi support on Flink-CDC.

Summary and Changelog

Changing access level and adding correspondent getter/setters to allow required fields to be accessed from wrapper functions.

Impact

None.

Risk Level

None

Documentation Update

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 follow the
    instruction to make changes to the website. -->

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@voonhous voonhous requested a review from danny0405 October 24, 2025 04:07
@voonhous
Copy link
Copy Markdown
Member Author

@cshuo Upstreaming the pre-requisite changes here.

@github-actions github-actions Bot added the size:S PR with lines of changes in (10, 100] label Oct 24, 2025
@voonhous voonhous force-pushed the flink-cdc-preqrequisite branch from 1d7d9ad to a6e8de0 Compare October 24, 2025 07:20
@hudi-bot
Copy link
Copy Markdown
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@danny0405 danny0405 merged commit ba1db10 into apache:master Oct 27, 2025
69 of 70 checks passed
@yihua yihua added this to the release-1.1.0 milestone Oct 27, 2025
@voonhous voonhous deleted the flink-cdc-preqrequisite branch December 11, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-1.1.0 size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants