Hi Seung-chan, Thanks for the KIP! I like the optimization to have a close method to force leave group. Some minor comments: 1. in the "public interfaces", there are many `close` method listed, but I believe you only want to introduce the one with `CloseOptions`, right? Could you add some comment to it to point it out this is what you added in this KIP? 2. In the CloseOptions class variable, I saw there's a `timeoutMs` as type `Integer`. Could you explain why we use `Integer` here, instead of `int`, or `Duration`?
Thank you. Luke On Tue, Feb 15, 2022 at 11:19 PM Seung-chan Ahn <dev.issea1...@gmail.com> wrote: > Hi Guozhang, > > > Thanks a lot for sharing your idea and guiding me. > > > Thanks to your support, we've reached this voting: > > https://lists.apache.org/thread/toq5pg799ctd7lwdcd6g7zk6xn73h26r > > > Seung-chan > > On Wed, Jan 12, 2022 at 10:50 PM Seung-chan Ahn <dev.issea1...@gmail.com> > wrote: > > > Hi team, > > > > Here's the new KIP > > < > https://cwiki.apache.org/confluence/display/KAFKA/KIP-812%3A+Introduce+another+form+of+the+%60KafkaStreams.close%28%29%60+API+that+forces+the+member+to+leave+the+consumer+group > > > > for this issue <https://issues.apache.org/jira/browse/KAFKA-13217>. > > > > The title says pretty much what this KIP is for. Even though it's my > first > > draft, as A. Sophie Blee-Goldman has written rich descriptions and > already > > the solutions in the issue thread, I've enjoyed following up on the idea. > > > > Please feel free to review on any point! > > >
