Skip to content

Conversation

@voonhous
Copy link
Member

Describe the issue this Pull Request addresses

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

  • Added Lombok annotations wherever possible to hudi-timeline-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

@voonhous voonhous requested a review from yihua December 29, 2025 20:29
@github-actions github-actions bot added the size:M PR with lines of changes in (100, 300] label Dec 29, 2025
@voonhous voonhous force-pushed the lombokify-hudi-timeline-service branch from 6ec8f2a to 93e0f48 Compare January 3, 2026 19:04
@voonhous voonhous force-pushed the lombokify-hudi-timeline-service branch from 93e0f48 to 968e84f Compare January 3, 2026 19:06
@hudi-bot
Copy link
Collaborator

hudi-bot commented Jan 3, 2026

CI report:

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

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.

2 participants