voonhous opened a new pull request, #18944:
URL: https://github.com/apache/hudi/pull/18944

   ### Describe the issue this Pull Request addresses
   
   This PR continues the effort to reduce boilerplate in the `hudi-common` 
module by
   applying [Project Lombok](https://projectlombok.org/) annotations. It 
replaces manual
   `Logger` instantiation, hand-written getters/setters, `equals`/`hashCode`, 
`toString`,
   and empty constructors with their Lombok equivalents.
   
   Pure refactor: no runtime logic changes.
   
   Adding Lombok to `hudi-common` is split into multiple parts to keep each PR 
reviewable.
   This is **part 7** and is scoped to:
   
   - `hudi-common/.../common/table/` (config, meta client, version, schema 
resolver)
   - `hudi-common/.../common/table/cdc/`
   - `hudi-common/.../common/table/checkpoint/`
   - `hudi-common/.../common/table/log/` and `.../log/block/`
   - `hudi-common/.../common/table/read/` and `.../read/buffer/`
   
   ### Summary and Changelog
   
   Applies Lombok annotations across **28 classes** under `common/table/`, 
removing
   hand-written boilerplate (net **-351** lines: 282 insertions, 633 deletions).
   
   Annotations used:
   
   | Annotation | Replaces |
   | --- | --- |
   | `@Slf4j` | manual `Logger LOG = LoggerFactory.getLogger(...)` |
   | `@Getter` / `@Setter` | manual accessors (with 
`AccessLevel.NONE`/`PROTECTED` where scoped) |
   | `@AllArgsConstructor` / `@NoArgsConstructor` | explicit constructors |
   | `@EqualsAndHashCode(onlyExplicitlyIncluded = true)` | hand-written 
`equals`/`hashCode` |
   | `@ToString(onlyExplicitlyIncluded = true)` | hand-written `toString` |
   | `@Accessors(fluent = true)`, `@NonNull` | accessor style / null checks |
   
   ### Impact
   
   - **Public API:** None.
   - **User experience:** No visible change for end users.
   - **Performance:** No impact (compile-time code generation).
   - **Code health:** Fewer lines, standardized logging and accessor patterns.
   
   ### Risk Level
   
   `none`
   
   Pure refactor using standard Lombok annotations; no business logic was 
modified.
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Enough context is provided in the sections above
   - [x] Adequate tests were added if applicable
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to