Skip to content

refactor: Add Lombok annotations to hudi-platform-service#17719

Merged
yihua merged 2 commits intoapache:masterfrom
voonhous:lombokify_hudi-platform-service
Dec 26, 2025
Merged

refactor: Add Lombok annotations to hudi-platform-service#17719
yihua merged 2 commits intoapache:masterfrom
voonhous:lombokify_hudi-platform-service

Conversation

@voonhous
Copy link
Member

Describe the issue this Pull Request addresses

This PR refactors the hudi-platform-service/* 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-platform-service/*modules and refactors several classes to utilize Lombok annotations.

  • Added Lombok annotations wherever possible to hudi-platform-service/* 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

@github-actions github-actions bot added the size:M PR with lines of changes in (100, 300] label Dec 26, 2025
@apache apache deleted a comment from hudi-bot Dec 26, 2025
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

@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

@yihua yihua merged commit 6e38eaf into apache:master Dec 26, 2025
713 of 762 checks passed
@voonhous voonhous deleted the lombokify_hudi-platform-service branch December 26, 2025 17:11
PavithranRick pushed a commit to PavithranRick/hudi that referenced this pull request Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M PR with lines of changes in (100, 300]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants