Hi, Yes, we can assign new id to this KIP.
Thanks. On Tue, Jun 30, 2020 at 6:59 PM Dániel Urbán <urb.dani...@gmail.com> wrote: > Hi, > > To help with the discussion, I also have a PR for this KIP now. reflecting > the current state of the KIP: https://github.com/apache/kafka/pull/8957. > I would like to ask a committer to start the test job on it. > > One thing I realised though is that there is a KIP id collision, there is > another KIP with the same id: > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=85474993 > What is the protocol in this case? Should I acquire a new id for the > GetOffsetShell KIP, and update it? > > Thanks in advance, > Daniel > > Dániel Urbán <dur...@cloudera.com.invalid> ezt írta (időpont: 2020. jún. > 30., K, 9:23): > > > Hi Manikumar, > > > > Thanks for the comments. > > 1. Will change this - thought that "command-config" is used for admin > > clients. > > 2. It's not necessary, just felt like a nice quality-of-life feature - > will > > remove it. > > > > Thanks, > > Daniel > > > > On Tue, Jun 30, 2020 at 4:16 AM Manikumar <manikumar.re...@gmail.com> > > wrote: > > > > > Hi Daniel, > > > > > > Thanks for working on this KIP. Proposed changes looks good to me, > > > > > > minor comments: > > > 1. We use "command-config" option name in most of the cmdline tools to > > pass > > > config > > > properties file. We can use the same name here. > > > > > > 2. Not sure, if we need a separate option to pass an consumer property. > > > fewer options are better. > > > > > > Thanks, > > > Manikumar > > > > > > On Wed, Jun 24, 2020 at 8:53 PM Dániel Urbán <urb.dani...@gmail.com> > > > wrote: > > > > > > > Hi, > > > > > > > > I see that this KIP turned somewhat inactive - I'd like to pick it up > > and > > > > work on it if it is okay. > > > > Part of the work is done, as switching to the Consumer API is already > > in > > > > trunk, but some functionality is still missing. > > > > > > > > I've seen the current PR and the discussion so far, only have a few > > > things > > > > to add: > > > > - I like the idea of the topic-partition argument, it would be useful > > to > > > > filter down to specific partitions. > > > > - Instead of a topic list arg, a pattern would be more powerful, and > > also > > > > fit better with the other tools (e.g. how the kafka-topics tool > works). > > > > > > > > Regards, > > > > Daniel > > > > > > > > > >
