Hi Yishun, Thanks for looking at this.
Under "proposed changes," it's not necessary to add a section where you demonstrate adding "implements AutoCloseable" to the code. We know what adding that would look like. Can you create a full, single, list of all the classes that would be affected? It's not necessary to write who suggested which classes in the KIP. Also, I noticed some of the classes here are in "internals" packages. Given that these are internal classes that aren't part of our API, it's not necessary to add them to the KIP, I think. Since they are implementation details, they can be changed at any time without a KIP. The "compatibility" section should have a discussion of the fact that we can add the new interface without requiring any backwards-incompatible changes at the source or binary level. In particular, it would be good to highlight that we are not renaming or changing the existing "close" methods. Under "rejected alternatives" we could explain why we chose to implement AutoCloseable rather than Closeable. cheers, Colin On Thu, Oct 11, 2018, at 13:48, Yishun Guan wrote: > Hi, > > Just to bump this voting thread up again. Thanks! > > Best, > Yishun > On Fri, Oct 5, 2018 at 12:58 PM Yishun Guan <gyis...@gmail.com> wrote: > > > > Hi, > > > > I think we have discussed this well enough to put this into a vote. > > > > Suggestions are welcome! > > > > Best, > > Yishun > > > > On Wed, Oct 3, 2018, 2:30 PM Yishun Guan <gyis...@gmail.com> wrote: > >> > >> Hi All, > >> > >> I want to start a voting on this KIP: > >> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=93325308 > >> > >> Here is the discussion thread: > >> https://lists.apache.org/thread.html/9f6394c28d3d11a67600d5d7001e8aaa318f1ad497b50645654bbe3f@%3Cdev.kafka.apache.org%3E > >> > >> Thanks, > >> Yishun