ZephyrYWZhou opened a new pull request, #15919: URL: https://github.com/apache/iceberg/pull/15919
### What Update test classes and methods under `core/src/test/.../util/` to use package-private visibility by removing unnecessary `public` modifiers. ### Why JUnit 5 does not require test classes or methods to be public, and the project convention recommends package-private visibility unless broader access is required. These 25 test files in the core util package still used `public` visibility and were updated to align with the convention. ### Notes - `FakeTicker` remains public because it is referenced by test classes in other packages (`hadoop`, `rest`). - Methods in inner classes implementing interfaces (e.g., in `TestLockManagers`) remain public as required by the interface contract. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
