Hi. I request the community to kindly resume the voting process for KIP-578. If you have further comments, we can address those as well.
Thanks and cheers. On Tue, Jul 21, 2020 at 2:07 PM Gokul Ramanan Subramanian < gokul24...@gmail.com> wrote: > Hi. > > Can we resume the voting process for KIP-578? I have addressed additional > comments by Boyang and Ismael. > > Thanks. > > On Mon, Jun 8, 2020 at 9:09 AM Gokul Ramanan Subramanian < > gokul24...@gmail.com> wrote: > >> Hi. Can we resume the voting process for KIP-578? Thanks. >> >> On Mon, Jun 1, 2020 at 11:09 AM Gokul Ramanan Subramanian < >> gokul24...@gmail.com> wrote: >> >>> Thanks Colin. Have updated the KIP per your recommendations. Let me know >>> what you think. >>> >>> Thanks Harsha for the vote. >>> >>> On Wed, May 27, 2020 at 8:17 PM Colin McCabe <cmcc...@apache.org> wrote: >>> >>>> Hi Gokul Ramanan Subramanian, >>>> >>>> Thanks for the KIP. >>>> >>>> Can you please modify the KIP to remove the reference to the deprecated >>>> --zookeeper flag? This is not how kafka-configs.sh is supposed to be used >>>> in new versions of Kafka. You get a warning message if you do use this >>>> deprecated flag. As described in KIP-604, we are removing the --zookeeper >>>> flag in the Kafka 3.0 release. It also causes problems when people use the >>>> deprecated access mode-- for example, as you note in this KIP, it bypasses >>>> resource limits such as the ones described here. >>>> >>>> Instead of WILL_EXCEED_PARTITION_LIMITS, how about >>>> RESOURCE_LIMIT_REACHED? Then the error string can contain the detailed >>>> message about which resource limit was hit (per broker limit, per cluster >>>> limit, whatever.) It would also be good to spell out that >>>> CreateTopicsPolicy plugins can also throw this exception, for consistency. >>>> >>>> I realize that 2 billion partitions seems like a very big number. >>>> However, filesystems have had to transition to 64 bit inode numbers as time >>>> has gone on. There doesn't seem to be any performance reason why this >>>> should be a 31 bit number, so let's just make these configurations longs, >>>> not ints. >>>> >>>> best, >>>> Colin >>>> >>>> >>>> On Wed, May 27, 2020, at 09:48, Harsha Chintalapani wrote: >>>> > Thanks for the KIP Gokul. This will be really useful for our use >>>> cases as >>>> > well. >>>> > +1 (binding). >>>> > >>>> > -Harsha >>>> > >>>> > >>>> > On Tue, May 26, 2020 at 12:33 AM, Gokul Ramanan Subramanian < >>>> > gokul24...@gmail.com> wrote: >>>> > >>>> > > Hi. >>>> > > >>>> > > Any votes for this? >>>> > > >>>> > > Thanks. >>>> > > >>>> > > On Tue, May 12, 2020 at 11:36 AM Gokul Ramanan Subramanian < >>>> gokul2411s@ >>>> > > gmail.com> wrote: >>>> > > >>>> > > Hello, >>>> > > >>>> > > I'd like to initialize voting on KIP-578: >>>> > > https://cwiki.apache.org/confluence/display/KAFKA/ >>>> > > KIP-578%3A+Add+configuration+to+limit+number+of+partitions >>>> > > . >>>> > > >>>> > > Got some good feedback from Stanislav Kozlovski, Alexandre Dupriez >>>> and Tom >>>> > > Bentley on the discussion thread. I have addressed their comments. >>>> I want >>>> > > to thank them for their time. >>>> > > >>>> > > If there are any more concerns about the KIP, I am happy to discuss >>>> them >>>> > > further. >>>> > > >>>> > > Thanks. >>>> > > >>>> > > >>>> > >>>> >>>