spotlesscoder opened a new pull request, #21608:
URL: https://github.com/apache/kafka/pull/21608
I think it is not necessary to pass the bootstrapServers separately from the
config object.
The class constructors have so many parameters that it makes the classes
quite hard to understand and use.
As the config is passed down into the call stack anyways, we can just obtain
the list of bootstrap servers right before we actually need it.
Note that I was not able to run the `gradlew test` target because my
resources are quite constrained and it overloads my PC so much that it freezes.
I also tried tried running with
```
./gradlew test \
-PmaxParallelForks=2 \
--max-workers=2
```
inside a VirtualBox VM with 10 Cores (i7 13700H on host) and 24 GB of memory
assigned to the VM - the entire VM crashes
--
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]
