hi PoAn, chia_q0: To prevent potential misinterpretations, could you please explicitly list the features that will be deprecated and added in Kafka 4.x, and those that will be removed in Kafka 5.x? Without this clarification, readers might mistakenly assume that breaking changes are planned for Kafka 4.x.
chia_q1: Furthermore, regarding the Endpoint.listenerName() method, our previous discussion [0] indicated that it returns an Optional value because it was intended for use by other public APIs. However, it appears that we have not fully utilized this approach. For instance, RaftVoterEndpoint [1] was created as a separate class to represent the endpoint for voters, rather than reusing the Endpoint class. I believe this discrepancy should be included in the KIP as a key motivation for the proposed changes. [0] https://lists.apache.org/thread/v6s7wnmzbprs26dyd7gom0g55k0ww1o2 [1] https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/admin/RaftVoterEndpoint.java Best, Chia-Ping Ismael Juma <m...@ismaeljuma.com> 於 2025年3月17日 週一 上午4:48寫道： > Hi, > > We cannot make incompatible changes to public classes outside of major > releases. The evolving annotation was left by mistake given that these > classes have been there for a long time by now. > > The general way to handle this is to have a new non deprecated method, > deprecate the old one. And then remove the deprecated one in 5.0. > > Ismael > > On Sun, Mar 16, 2025, 5:53 AM PoAn Yang <yangp...@gmail.com> wrote: > > > Hi all, > > > > I would like to start a discussion thread on KIP-1143. > > > > This KIP removes redundant org.apache.kafka.network.EndPoint and replaces > > it with org.apache.kafka.common.Endpoint. Also, it changes output of > > Endpoint#listenerName from Optional<String> to String. > > > > Please take a look and feel free to share any thoughts. > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=345377327 > > > > Sample PR: https://github.com/apache/kafka/pull/19191 > > > > Thanks, > > PoAn >
