Hi Seung-chan, Thanks for the update.
One more comment: 3. Could you explain if user doesn't provide the `timeout` value in `CloseOptions`. What will happen? I saw the default value for `timeout` variable in `CloseOptions` is `null`. What doest null `timeout` mean? Thank you. Luke On Sat, Feb 19, 2022 at 3:31 PM Seung-chan Ahn <dev.issea1...@gmail.com> wrote: > 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! > > > > > > > > > >
