divijvaidya commented on code in PR #15277:
URL: https://github.com/apache/kafka/pull/15277#discussion_r1472892693
##
gradle/dependencies.gradle:
##
@@ -234,6 +227,7 @@ libs += [
kafkaStreams_34: "org.apache.kafka:kafka-streams:$versions.kafka_34",
kafkaStreams_35: "org.ap
OmniaGM commented on code in PR #15277:
URL: https://github.com/apache/kafka/pull/15277#discussion_r1471541316
##
core/src/test/scala/kafka/security/minikdc/MiniKdc.scala:
##
@@ -118,123 +96,37 @@ class MiniKdc(config: Properties, workDir: File) extends
Logging {
throw n
mimaison commented on PR #15277:
URL: https://github.com/apache/kafka/pull/15277#issuecomment-1914302395
It looks like something is still not quite right, the latest [CI
run](https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15277/4/testReport/)
has many failures with:
```
[2
highluck commented on PR #15277:
URL: https://github.com/apache/kafka/pull/15277#issuecomment-1913643599
@mimaison
oh! thanks
that was fixed!
--
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 t
mimaison commented on PR #15277:
URL: https://github.com/apache/kafka/pull/15277#issuecomment-1913331999
Thanks for the PR!
This seems like a nice cleanup but this breaks a number of tests. See
https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15277/2/testReport/
for the tes
