ZaheerAhmadDev opened a new pull request, #8505:
URL: https://github.com/apache/hbase/pull/8505

   **Description:**
   
   Removes a couple of unused private fields identified while testing a new 
checkstyle check `UnusedPrivateFieldCheck`, currently in review upstream in the 
[checkstyle](https://github.com/checkstyle/checkstyle/pull/19816#issuecomment-5077926190)
 project.
   
   **Diff report of hbase :** Found  violations in diff report : 
https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/31a666b_2026091037/reports/diff/Hbase/index.html
   
   **What the check does:**
    it flags private fields that are declared but never referenced anywhere in 
the file  a common source of dead code in large, long-lived codebases. It's 
aware of common false-positive sources (reflection-based frameworks, Lombok, DI 
annotations, etc. via a configurable `ignoreAnnotationCanonicalNames` property) 
and ignores well-known name-based conventions like serialVersionUID by default 
via `ignoredFieldNames` or user can customize for their own use case .
   
   Each was manually verified as genuinely unused before removal  not 
referenced via reflection, annotations, or any other indirect means.
   
   This is primarily meant as real-world signal for the check development 
feedback on false positives, edge cases, or naming conventions this project 
relies on is very welcome, and will help shape the check before it's finalized 
upstream.


-- 
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