It needs the 3rd binding +1 yet. Could anyone else take a look? Thanks, Yunze
On Wed, Dec 21, 2022 at 3:21 PM 丛搏 <bog...@apache.org> wrote: > > Hi Yunze, > > add `TopicMessageId ` will couple messageID and `topic name` together, > which is very unclear for non-partition-topic. > > ``` > void seek(String topicName, MessageId messageId) throws PulsarClientException; > List<Map<String, MessageId>> getLastTopicMessageId() throws > PulsarClientException; > ``` > If the interface is designed in this way, it may be simpler, easier to > understand, and more intuitive for users, and MessageID will not be > coupled with TopicName. > > Thanks, > Bo > > Yunze Xu <y...@streamnative.io.invalid> 于2022年12月16日周五 15:31写道： > > > > Yeah, it's an implementation detail and I will keep the same semantics > > with the latest master when I push my PR. > > > > Thanks, > > Yunze > > > > On Fri, Dec 16, 2022 at 3:03 PM 丛搏 <bog...@apache.org> wrote: > > > > > > 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 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
