Skip to content

refactor: Add Lombok annotations to hudi-example modules#17589

Merged
yihua merged 1 commit intoapache:masterfrom
voonhous:hudi-examples-lombokify
Dec 14, 2025
Merged

refactor: Add Lombok annotations to hudi-example modules#17589
yihua merged 1 commit intoapache:masterfrom
voonhous:hudi-examples-lombokify

Conversation

@voonhous
Copy link
Member

Describe the issue this Pull Request addresses

This PR refactors the hudi-example modules to reduce boilerplate code by leveraging Project Lombok annotations. Specifically, it replaces explicit Logger instantiation, manual getter/setter methods, and empty constructors with their equivalent Lombok annotations (@Slf4j, @Getter, @Setter,@NoArgsConstructor, @AllArgsConstructor, @Data, @Value, @ToString).

This improves code readability and maintainability without altering the runtime logic.

Summary and Changelog

This change introduces the Lombok dependency to the hudi-example modules and refactors several classes to utilize Lombok annotations.

  • Added lombok annotations wherever possible to hudi-example modules.

Impact

  • Public API: None.
  • User Experience: No visible change for end-users.
  • Performance: No impact (compile-time code generation).
  • Code Health: Reduces lines of code and standardizes logging/accessor patterns.

Risk Level

none

(This is a pure refactoring change involving standard library annotations; no business logic was modified.)

Documentation Update

none

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 yihua December 13, 2025 19:21
@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Dec 13, 2025
@hudi-bot
Copy link
Collaborator

CI report:

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

Copy link
Contributor

@yihua yihua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yihua yihua merged commit fddd285 into apache:master Dec 14, 2025
136 of 137 checks passed
@voonhous voonhous deleted the hudi-examples-lombokify branch December 14, 2025 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants