UladzislauBlok opened a new pull request, #21289: URL: https://github.com/apache/kafka/pull/21289
`CoreUtils.scala` is currently a blocker for migrating `KafkaConfig.scala` to the server-common module. This PR introduces a Java implementation of `CoreUtils` within server-common to facilitate the ongoing effort of decoupling the server modules. **Key Changes:** - Rewrote the majority of utility methods from `CoreUtils.scala` into a new `CoreUtils.java` class, located in the `org.apache.kafka.server.util` package inside `server-common` module. - Moved `SocketServerConfigs` to the server-common module, as it is a required dependency for the migrated utility methods. - Note that lock-related methods were excluded from this migration as they were previously moved to `LockUtils` . see [PR #19961](https://github.com/apache/kafka/pull/19961). With the adjustments for read/write locks. **Architecture Note**: I have moved `SocketServerConfigs` to server-common to resolve dependency issues. Currently, configuration classes are split between server and server-common. I'd like to clarify **what is the strategic place** to keep config related classes? **Migration Strategy**: To keep the scope of this PR manageable and ensure a smoother review process, `CoreUtils.scala` has not been removed. This PR focuses on providing the new Java-based infrastructure. A subsequent PR will be submitted to replace existing call sites and perform the final cleanup of the Scala implementation. -- 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]
