You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by Xiangying Meng <xi...@apache.org> on 2023/09/27 07:04:43 UTC
[VOTE] PIP-302 Introduce refreshAsync API for TableView
Hi dev,
This thread is to start a vote for PIP-302 Add new API
refreshAsync for TableView.
Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
PIP: https://github.com/apache/pulsar/pull/21166
BR,
Xiangying
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by Baodi Shi <ba...@apache.org>.
+1 (no-binding)
Thanks,
Baodi Shi
On Oct 7, 2023 at 21:06:35, Yubiao Feng <yu...@streamnative.io.invalid>
wrote:
> +1 (no-binding)
>
> And correct the link of PIP:
> - https://github.com/apache/pulsar/pull/21271
>
> Thanks
> Yubiao Feng
>
> On Wed, Sep 27, 2023 at 3:04 PM Xiangying Meng <xi...@apache.org>
> wrote:
>
> Hi dev,
>
> This thread is to start a vote for PIP-302 Add new API
>
> refreshAsync for TableView.
>
> Discuss thread:
>
> https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
>
> PIP: https://github.com/apache/pulsar/pull/21166
>
>
> BR,
>
> Xiangying
>
>
>
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by Yubiao Feng <yu...@streamnative.io.INVALID>.
+1 (no-binding)
And correct the link of PIP:
- https://github.com/apache/pulsar/pull/21271
Thanks
Yubiao Feng
On Wed, Sep 27, 2023 at 3:04 PM Xiangying Meng <xi...@apache.org> wrote:
> Hi dev,
> This thread is to start a vote for PIP-302 Add new API
> refreshAsync for TableView.
> Discuss thread:
> https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> PIP: https://github.com/apache/pulsar/pull/21166
>
> BR,
> Xiangying
>
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by Dezhi Liu <de...@apache.org>.
+1
Thanks,
Dezhi
On 2023/09/27 07:04:43 Xiangying Meng wrote:
> Hi dev,
> This thread is to start a vote for PIP-302 Add new API
> refreshAsync for TableView.
> Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> PIP: https://github.com/apache/pulsar/pull/21166
>
> BR,
> Xiangying
>
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by Xiangying Meng <xi...@apache.org>.
Close this vote with 3 +1(binding)
- Penghui
- Jiwei Guo (Tboy)
- Mattison
On Sun, Oct 8, 2023 at 5:02 PM mattison chao <ma...@gmail.com> wrote:
>
> +1(binding)
>
> Best,
> Mattison
> On 8 Oct 2023 at 14:49 +0800, Yunze Xu <xy...@apache.org>, wrote:
> > Totally I'm +0 at the moment. I'm still wondering which issue you
> > really want to resolve. I left a comment
> > https://github.com/apache/pulsar/pull/21271#issuecomment-1751899833.
> > Generally you can get latest value unless the producer is far quicker
> > than the reader. However, even with the refreshAsync() method, in the
> > future's callback, the data could still be updated.
> >
> > From my perspective, this proposal only makes sense when you can
> > guarantee there is no more data written to the topic when you call
> > methods of TableView. But if so, a simple `boolean hasReachedLatest()`
> > method could do the trick.
> >
> > Thanks,
> > Yunze
> >
> > On Sun, Oct 8, 2023 at 2:12 PM 太上玄元道君 <da...@apache.org> wrote:
> > >
> > > +1 (no-binding)
> > >
> > >
> > > Xiangying Meng <xi...@apache.org>于2023年9月27日 周三15:05写道:
> > >
> > > > > Hi dev,
> > > > > This thread is to start a vote for PIP-302 Add new API
> > > > > refreshAsync for TableView.
> > > > > Discuss thread:
> > > > > https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > > > > PIP: https://github.com/apache/pulsar/pull/21166
> > > > >
> > > > > BR,
> > > > > Xiangying
> > > > >
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by mattison chao <ma...@gmail.com>.
+1(binding)
Best,
Mattison
On 8 Oct 2023 at 14:49 +0800, Yunze Xu <xy...@apache.org>, wrote:
> Totally I'm +0 at the moment. I'm still wondering which issue you
> really want to resolve. I left a comment
> https://github.com/apache/pulsar/pull/21271#issuecomment-1751899833.
> Generally you can get latest value unless the producer is far quicker
> than the reader. However, even with the refreshAsync() method, in the
> future's callback, the data could still be updated.
>
> From my perspective, this proposal only makes sense when you can
> guarantee there is no more data written to the topic when you call
> methods of TableView. But if so, a simple `boolean hasReachedLatest()`
> method could do the trick.
>
> Thanks,
> Yunze
>
> On Sun, Oct 8, 2023 at 2:12 PM 太上玄元道君 <da...@apache.org> wrote:
> >
> > +1 (no-binding)
> >
> >
> > Xiangying Meng <xi...@apache.org>于2023年9月27日 周三15:05写道:
> >
> > > > Hi dev,
> > > > This thread is to start a vote for PIP-302 Add new API
> > > > refreshAsync for TableView.
> > > > Discuss thread:
> > > > https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > > > PIP: https://github.com/apache/pulsar/pull/21166
> > > >
> > > > BR,
> > > > Xiangying
> > > >
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by Yunze Xu <xy...@apache.org>.
Totally I'm +0 at the moment. I'm still wondering which issue you
really want to resolve. I left a comment
https://github.com/apache/pulsar/pull/21271#issuecomment-1751899833.
Generally you can get latest value unless the producer is far quicker
than the reader. However, even with the refreshAsync() method, in the
future's callback, the data could still be updated.
From my perspective, this proposal only makes sense when you can
guarantee there is no more data written to the topic when you call
methods of TableView. But if so, a simple `boolean hasReachedLatest()`
method could do the trick.
Thanks,
Yunze
On Sun, Oct 8, 2023 at 2:12 PM 太上玄元道君 <da...@apache.org> wrote:
>
> +1 (no-binding)
>
>
> Xiangying Meng <xi...@apache.org>于2023年9月27日 周三15:05写道:
>
> > Hi dev,
> > This thread is to start a vote for PIP-302 Add new API
> > refreshAsync for TableView.
> > Discuss thread:
> > https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > PIP: https://github.com/apache/pulsar/pull/21166
> >
> > BR,
> > Xiangying
> >
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by 太上玄元道君 <da...@apache.org>.
+1 (no-binding)
Xiangying Meng <xi...@apache.org>于2023年9月27日 周三15:05写道:
> Hi dev,
> This thread is to start a vote for PIP-302 Add new API
> refreshAsync for TableView.
> Discuss thread:
> https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> PIP: https://github.com/apache/pulsar/pull/21166
>
> BR,
> Xiangying
>
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by guo jiwei <te...@apache.org>.
+1
Regards
Jiwei Guo (Tboy)
On Sat, Oct 7, 2023 at 8:07 PM PengHui Li <pe...@apache.org> wrote:
> +1
>
> Thanks,
> Penghui
>
> On Wed, Sep 27, 2023 at 3:09 PM Zike Yang <zi...@apache.org> wrote:
>
> > +1
> >
> > BR,
> > Zike Yang
> >
> > On Wed, Sep 27, 2023 at 3:05 PM Xiangying Meng <xi...@apache.org>
> > wrote:
> > >
> > > Hi dev,
> > > This thread is to start a vote for PIP-302 Add new API
> > > refreshAsync for TableView.
> > > Discuss thread:
> > https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > > PIP: https://github.com/apache/pulsar/pull/21166
> > >
> > > BR,
> > > Xiangying
> >
>
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by PengHui Li <pe...@apache.org>.
+1
Thanks,
Penghui
On Wed, Sep 27, 2023 at 3:09 PM Zike Yang <zi...@apache.org> wrote:
> +1
>
> BR,
> Zike Yang
>
> On Wed, Sep 27, 2023 at 3:05 PM Xiangying Meng <xi...@apache.org>
> wrote:
> >
> > Hi dev,
> > This thread is to start a vote for PIP-302 Add new API
> > refreshAsync for TableView.
> > Discuss thread:
> https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > PIP: https://github.com/apache/pulsar/pull/21166
> >
> > BR,
> > Xiangying
>
Re: [VOTE] PIP-302 Introduce refreshAsync API for TableView
Posted by Zike Yang <zi...@apache.org>.
+1
BR,
Zike Yang
On Wed, Sep 27, 2023 at 3:05 PM Xiangying Meng <xi...@apache.org> wrote:
>
> Hi dev,
> This thread is to start a vote for PIP-302 Add new API
> refreshAsync for TableView.
> Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> PIP: https://github.com/apache/pulsar/pull/21166
>
> BR,
> Xiangying