fvaleri commented on code in PR #13127:
URL: https://github.com/apache/kafka/pull/13127#discussion_r1081230571
##########
build.gradle:
##########
@@ -1757,6 +1757,7 @@ project(':tools') {
archivesBaseName = "kafka-tools"
dependencies {
+ implementation project(':core')
Review Comment:
@cadonna yes, there is some confusion on which tools need to be migrated.
For example, `DumpLogSegments` has multiple dependencies on `core` classes, so
it should be excluded, but was added as a sub task. Checking with @mimaison how
we want to precede with these. All the rest can be migrated.
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
