if you don't change this in PIP-229 or PIP-224, I will create a new PIP to handle the `BatchMessageIdImpl` and `MessageIdImpl` `compareTo()` method, now I have no problem with this PIP +1 (non-binding) Sorry to bother this PIP vote.
Thanks, Bo Yunze Xu <y...@streamnative.io.invalid> 于2022年12月16日周五 11:58写道： > > If this breaking change can pass the PMC votes, I will keep the new > semantics in PIP-229. Otherwise, it would not make sense to adopt the > new semantics in PIP-229. > > Thanks, > Yunze > > On Fri, Dec 16, 2022 at 11:46 AM Yunze Xu <y...@streamnative.io> wrote: > > > > I cannot find any confusing code from the proposal itself. Could you > > point it out? If you are mentioning the `legacyCompare` and `compare` > > methods in #18890 [1], it's not related to this proposal. And I have > > opened PIP-229 [2] for discussion. > > > > BTW, the PIP-229 itself doesn't mention the compare logic. But I'm not > > going to adopt the new semantics because it's actually a breaking > > change, just as I've replied. You might think it's a bug, but it's a > > public API. Any change of the semantics in the public API is a > > breaking change. > > > > [1] https://github.com/apache/pulsar/pull/18890/files > > [2] https://lists.apache.org/thread/x52zpwlo8pxzp81oxllh5vw82kyrzgpk > > > > On Fri, Dec 16, 2022 at 11:34 AM 丛搏 <congbobo...@gmail.com> wrote: > > > > > > Although unrelated, it adds a lot of confusing code. > > > > > > Thanks, > > > Bo > > > > > > Yunze Xu <y...@streamnative.io.invalid> 于2022年12月16日周五 08:05写道： > > > > > > > > > > > This proposal is not related to the comparison logic between > > > > BatchMessageIdImpl and MessageIdImpl. > > > > > > > > Thanks, > > > > Yunze > > > > > > > > On Thu, Dec 15, 2022 at 12:58 PM 丛搏 <bog...@apache.org> wrote: > > > > > > > > > > -1 (non-binding) > > > > > sorry, I have one question about the BatchMessageId compareTo() > > > > > method. the discussion mail : > > > > > https://lists.apache.org/thread/8n3oyk2hdsskkotnj4lnlvfnndctpqbg. > > > > > I hope it can be this issue can be discussed clearly. > > > > > > > > > > I hope it can be this issue can be discussed clearly. I will retry to > > > > > vote until this issue clearly : > > > > > > > > > > > > > > > Thanks, > > > > > Bo > > > > > > > > > > 丛搏 <congbobo...@gmail.com> 于2022年12月14日周三 22:56写道： > > > > > > > > > > > > +1 (non-binding) > > > > > > > > > > > > Thanks, > > > > > > Bo > > > > > > > > > > > > PengHui Li <peng...@apache.org> 于2022年12月14日周三 19:12写道： > > > > > > > > > > > > > > +1 (binding) > > > > > > > > > > > > > > - Penghui > > > > > > > > > > > > > > On Sun, Dec 11, 2022 at 6:36 AM Enrico Olivelli > > > > > > > <eolive...@gmail.com> wrote: > > > > > > > > > > > > > > > +1 (binding) > > > > > > > > > > > > > > > > Enrico > > > > > > > > > > > > > > > > Il Ven 9 Dic 2022, 10:41 Jiaqi Shen <gleiphir2...@gmail.com> ha > > > > > > > > scritto: > > > > > > > > > > > > > > > > > +1(non-binding) > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > Jiaqi Shen > > > > > > > > > > > > > > > > > > > > > > > > > > > <mattisonc...@gmail.com> 于2022年12月5日周一 15:23写道： > > > > > > > > > > > > > > > > > > > +1(non-binding) > > > > > > > > > > > > > > > > > > > > Best, > > > > > > > > > > Mattison > > > > > > > > > > On Dec 5, 2022, 15:09 +0800, Zike Yang <z...@apache.org>, > > > > > > > > > > wrote: > > > > > > > > > > > +1(non-binding) > > > > > > > > > > > > > > > > > > > > > > Best, > > > > > > > > > > > Zike Yang > > > > > > > > > > > > > > > > > > > > > > On Mon, Dec 5, 2022 at 2:41 PM Baodi Shi > > > > > > > > <baodi....@icloud.com.invalid > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > +1(non-binding) > > > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > Baodi Shi > > > > > > > > > > > > > > > > > > > > > > > > > > 2022年12月5日 12:51，Yunze Xu > > > > > > > > > > > > > > <y...@streamnative.io.INVALID> 写道： > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi all, > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'm starting the vote for PIP-224: Introduce > > > > > > > > > > > > > > TopicMessageId for > > > > > > > > > > > > > > consumer's MessageId related APIs: > > > > > > > > > > > > > > https://github.com/apache/pulsar/issues/18616 > > > > > > > > > > > > > > > > > > > > > > > > > > > > Here is the discussion thread: > > > > > > > > > > > > > > > > > > > > > > https://lists.apache.org/thread/jhqy65cdyxzmmxnfsjm8rv9pbk76noxy > > > > > > > > > > > > > > > > > > > > > > > > > > > > The vote will be open for at least 3 days. > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > Yunze > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
