Hi Luke, Thanks for reviewing this KIP!
1. You are right, I'm only adding one method with `CloseOptions`. Let me comment in the KIP! -> Done 2. I think it makes more sense to have `Duration` type option which is aligned to the existing public method. Let me update the KIP! -> Done Seung-chan On Fri, Feb 18, 2022 at 3:21 PM Luke Chen <show...@gmail.com> wrote: > 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! > > > > > >
