Thanks for the KIP Rajani.
`BrokerNotFoundException` is unused for a long time, and I am happy to
get rid of it.
However, I am not sure, if we can remove it directly -- at least, that
would not be regular protocol. We usually first deprecate
interfaces/classes/methods we want to remove, and remove them only in
the next major release (if they are deprecated for at least 3 releases /
1 year).
Given that `BrokerNotFoundException` is unused for a very long time, we
could make an exception and remove directly, but on the other hand,
there is also no damage if we only deprecate it for now, and wait for
5.0 release to remove it.
Curious to hear from others what they thing about this.
For the other methods from the exception handlers you list on the KIP,
all these are already deprecated. However, we could not remove them with
4.0 release, because they did not meet the 3-release/1-year grace period
for removal. Thus, we can only remove them with 5.0 release (we don't
need a KIP for this, as the removal is implicitly approved with the KIPs
which deprecated these methods).
-Matthias
On 7/16/25 2:36 AM, Rajani Karuturi wrote:
Hi All,
I would like to start a discussion for KIP-1195(
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=373886192)
Over time, certain exceptions and methods within the Kafka Streams
exception handling interfaces have been deprecated in favor of newer, more
comprehensive alternatives. Retaining these deprecated elements can lead to
confusion, and complicates future development and maintenance. This KIP
proposes the removal of these specific deprecated exceptions and methods to
streamline the API and improve code clarity.
Thanks,
~ Rajani
