Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
Thanks for voting. +1 (binding) for myself. I am closing this vote. The KIP is accepted with 4 binding (Gouzhang, Bill, Gwen, Matthias) and 2 non-binding (John, Bruno) votes. -Matthias On 9/20/19 10:41 AM, Gwen Shapira wrote: > +1 (binding) > > On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax wrote: >> >> Hi, >> >> I would like to propose a small KIP to deprecate some public APIs that >> are considered "dangerous" to use, with the goal to remove them in the >> next major release. >> >> Because the KIP is straight forward and to make it into 2.4 release, I >> call directly for a vote. >> >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface >> >> >> -Matthias >> signature.asc Description: OpenPGP digital signature
Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
+1 (binding) On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax wrote: > > Hi, > > I would like to propose a small KIP to deprecate some public APIs that > are considered "dangerous" to use, with the goal to remove them in the > next major release. > > Because the KIP is straight forward and to make it into 2.4 release, I > call directly for a vote. > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface > > > -Matthias >
Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
Hi Matthias, +1 (non-binding) Best, Bruno On Fri, Sep 20, 2019 at 10:02 AM Matthias J. Sax wrote: > > Sure -- don't think we need to cover it in the KIP though. > > Btw: an implicit deprecation is implied by this KIP for the currently > public default implementation of the `PartitionGrouper` interface. When > we finally remove the public interface, we can move the default > implementation into an internal package (cf > https://issues.apache.org/jira/browse/KAFKA-7785) > > > -Matthias > > On 9/20/19 9:49 AM, John Roesler wrote: > > +1 (non-binding). Thanks for bringing this up. > > > > Should we also log a deprecation warning on startup if the config is > > used? It would be just one extra opportunity to learn of the > > deprecation in case you missed the compiler's warning. > > > > -John > > > > On Thu, Sep 19, 2019 at 4:49 PM Bill Bejeck wrote: > >> > >> Thanks for the KIP Matthias. I agree with Guozhang on this one. > >> > >> +1 (binding) > >> > >> -Bill > >> > >> On Thu, Sep 19, 2019 at 4:41 PM Guozhang Wang wrote: > >> > >>> +1, we should do that long ago :) > >>> > >>> Guozhang > >>> > >>> On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax > >>> wrote: > >>> > Hi, > > I would like to propose a small KIP to deprecate some public APIs that > are considered "dangerous" to use, with the goal to remove them in the > next major release. > > Because the KIP is straight forward and to make it into 2.4 release, I > call directly for a vote. > > > > >>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface > > > -Matthias > > > >>> > >>> -- > >>> -- Guozhang > >>> >
Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
Sure -- don't think we need to cover it in the KIP though. Btw: an implicit deprecation is implied by this KIP for the currently public default implementation of the `PartitionGrouper` interface. When we finally remove the public interface, we can move the default implementation into an internal package (cf https://issues.apache.org/jira/browse/KAFKA-7785) -Matthias On 9/20/19 9:49 AM, John Roesler wrote: > +1 (non-binding). Thanks for bringing this up. > > Should we also log a deprecation warning on startup if the config is > used? It would be just one extra opportunity to learn of the > deprecation in case you missed the compiler's warning. > > -John > > On Thu, Sep 19, 2019 at 4:49 PM Bill Bejeck wrote: >> >> Thanks for the KIP Matthias. I agree with Guozhang on this one. >> >> +1 (binding) >> >> -Bill >> >> On Thu, Sep 19, 2019 at 4:41 PM Guozhang Wang wrote: >> >>> +1, we should do that long ago :) >>> >>> Guozhang >>> >>> On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax >>> wrote: >>> Hi, I would like to propose a small KIP to deprecate some public APIs that are considered "dangerous" to use, with the goal to remove them in the next major release. Because the KIP is straight forward and to make it into 2.4 release, I call directly for a vote. >>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface -Matthias >>> >>> -- >>> -- Guozhang >>> signature.asc Description: OpenPGP digital signature
Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
+1 (non-binding). Thanks for bringing this up. Should we also log a deprecation warning on startup if the config is used? It would be just one extra opportunity to learn of the deprecation in case you missed the compiler's warning. -John On Thu, Sep 19, 2019 at 4:49 PM Bill Bejeck wrote: > > Thanks for the KIP Matthias. I agree with Guozhang on this one. > > +1 (binding) > > -Bill > > On Thu, Sep 19, 2019 at 4:41 PM Guozhang Wang wrote: > > > +1, we should do that long ago :) > > > > Guozhang > > > > On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax > > wrote: > > > > > Hi, > > > > > > I would like to propose a small KIP to deprecate some public APIs that > > > are considered "dangerous" to use, with the goal to remove them in the > > > next major release. > > > > > > Because the KIP is straight forward and to make it into 2.4 release, I > > > call directly for a vote. > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface > > > > > > > > > -Matthias > > > > > > > > > > -- > > -- Guozhang > >
Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
Thanks for the KIP Matthias. I agree with Guozhang on this one. +1 (binding) -Bill On Thu, Sep 19, 2019 at 4:41 PM Guozhang Wang wrote: > +1, we should do that long ago :) > > Guozhang > > On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax > wrote: > > > Hi, > > > > I would like to propose a small KIP to deprecate some public APIs that > > are considered "dangerous" to use, with the goal to remove them in the > > next major release. > > > > Because the KIP is straight forward and to make it into 2.4 release, I > > call directly for a vote. > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface > > > > > > -Matthias > > > > > > -- > -- Guozhang >
Re: [VOTE] KIP-528: Deprecate PartitionGrouper configuration and interface
+1, we should do that long ago :) Guozhang On Thu, Sep 19, 2019 at 4:28 PM Matthias J. Sax wrote: > Hi, > > I would like to propose a small KIP to deprecate some public APIs that > are considered "dangerous" to use, with the goal to remove them in the > next major release. > > Because the KIP is straight forward and to make it into 2.4 release, I > call directly for a vote. > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-528%3A+Deprecate+PartitionGrouper+configuration+and+interface > > > -Matthias > > -- -- Guozhang