raveendra11 opened a new pull request, #7276: URL: https://github.com/apache/gravitino/pull/7276
**Title**: `[#7246] improvement(core): make abstract class constructors protected instead of public` ### What changes were proposed in this pull request? This PR updates the visibility of constructors in abstract classes from `public` to `protected`. This aligns with best practices, as abstract classes are not intended to be instantiated directly. Changes include: * Modifying all `public` constructors in abstract classes to `protected`. * Ensuring that no instantiation of these abstract classes is done outside their inheritance hierarchy. ### Why are the changes needed? Making constructors of abstract classes `protected` improves encapsulation and API clarity by: 1. Preventing misuse through direct instantiation. 2. Encouraging proper inheritance and design patterns. 3. Aligning with Java best practices and improving code maintainability. ### Fix: Fixes #7246 ### Does this PR introduce *any* user-facing change? No user-facing changes are introduced. This is an internal improvement in the codebase to follow Java best practices for abstract class design. ### How was this patch tested? * Verified that the build compiles without errors. -- 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]
