Hi,

I've just found that KafkaStreams.toString [1] is deprecated, but I think
that it does not make sense.

The parameterless toString is simply part of the java.lang.Object contract
and I don't think it could ever get deprecated (unless it is by Object).

I think it'd be much better if the method used whatever it's recommended
for a toString-like functionality (which seems that
KafkaStreams.localThreadsMetadata [2] is or something based on that).

Thoughts?

p.s. I also think that since KafkaStreams is marked
as @InterfaceStability.Evolving using @Deprecated markers does not add much
if anything. I thought that Evolving was to say that literally everything
could change without warning at any time, couldn't it?

[1]
https://github.com/apache/kafka/blob/trunk/streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java?utf8=%E2%9C%93#L900

[2]
https://github.com/apache/kafka/blob/trunk/streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java?utf8=%E2%9C%93#L1061

Pozdrawiam,
Jacek Laskowski
----
https://about.me/JacekLaskowski
Mastering Spark SQL https://bit.ly/mastering-spark-sql
Spark Structured Streaming https://bit.ly/spark-structured-streaming
Mastering Kafka Streams https://bit.ly/mastering-kafka-streams
Follow me at https://twitter.com/jaceklaskowski

Reply via email to