siddharthaDevineni commented on code in PR #21601:
URL: https://github.com/apache/kafka/pull/21601#discussion_r2869811676
##########
build.gradle:
##########
@@ -2046,6 +2046,7 @@ project(':clients') {
include "**/org/apache/kafka/common/security/scram/*"
include "**/org/apache/kafka/common/security/token/delegation/*"
include "**/org/apache/kafka/common/serialization/*"
+ include "**/org/apache/kafka/common/utils/Bytes.java"
Review Comment:
You are right, it seems a bit odd.
I will move the remaining classes, except Bytes and Time into the internals
package and add the utils package to the javadoc. This would be much cleaner.
Thanks for the input.
By the way, is it "internal" or "internals" because there is an "internals"
package under "org.apache.kafka.streams.state",
"org.apache.kafka.common.config", "org.apache.kafka.common.header",
"org.apache.kafka.common.metrics", and many more.
--
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]
