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/25 07:25:09 UTC
[VOTE] PIP-302 Add new API readAllExistingMessages for TableView
Hi dev,
This thread is to start a vote for PIP-302 Add new API
readAllExistingMessages for TableView.
Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
PIP: https://github.com/apache/pulsar/pull/21166
BR,
Xiangying
Re: [VOTE] PIP-302 Add new API readAllExistingMessages for TableView
Posted by Xiangying Meng <xi...@apache.org>.
Close this via https://lists.apache.org/thread/vox93tmj33mms026wt52l92h1wffctbk
On Mon, Sep 25, 2023 at 6:34 PM Xiangying Meng <xi...@apache.org> wrote:
>
> Thank you for your reminder. In our discussion, there were several
> changes to the specific plan and method names, which resulted in the
> PR title not being updated promptly. This was my oversight. The email
> title for the vote was not modified to match the titles of the
> discussed emails.
>
> Regarding my proposal itself, would you happen to have any other questions?
>
> BR,
> Xiangying
>
> On Mon, Sep 25, 2023 at 6:03 PM Zike Yang <zi...@apache.org> wrote:
> >
> > Hi, Xiangying
> >
> > This PIP is a little confusing to me.
> > This mail title states that we are introducing `readAllExistingMessages`.
> > But actually, we only introduced `refreshAsync` in the PIP:
> > https://github.com/apache/pulsar/pull/21166/files#diff-45c655583d6c0c73d87afd3df3fe67f77caadbf1bd691cf8f8211cc89728a1ceR34-R36
> > And the PR title doesn't seem relevant. “PIP-302 Add alwaysRefresh
> > Configuration Option for TableView to Read Latest Values”
> >
> > BR,
> > Zike Yang
> >
> > On Mon, Sep 25, 2023 at 3:25 PM Xiangying Meng <xi...@apache.org> wrote:
> > >
> > > Hi dev,
> > > This thread is to start a vote for PIP-302 Add new API
> > > readAllExistingMessages for TableView.
> > > Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > > PIP: https://github.com/apache/pulsar/pull/21166
> > >
> > > BR,
> > > Xiangying
Re: [VOTE] PIP-302 Add new API readAllExistingMessages for TableView
Posted by Xiangying Meng <xi...@apache.org>.
Thank you for your reminder. In our discussion, there were several
changes to the specific plan and method names, which resulted in the
PR title not being updated promptly. This was my oversight. The email
title for the vote was not modified to match the titles of the
discussed emails.
Regarding my proposal itself, would you happen to have any other questions?
BR,
Xiangying
On Mon, Sep 25, 2023 at 6:03 PM Zike Yang <zi...@apache.org> wrote:
>
> Hi, Xiangying
>
> This PIP is a little confusing to me.
> This mail title states that we are introducing `readAllExistingMessages`.
> But actually, we only introduced `refreshAsync` in the PIP:
> https://github.com/apache/pulsar/pull/21166/files#diff-45c655583d6c0c73d87afd3df3fe67f77caadbf1bd691cf8f8211cc89728a1ceR34-R36
> And the PR title doesn't seem relevant. “PIP-302 Add alwaysRefresh
> Configuration Option for TableView to Read Latest Values”
>
> BR,
> Zike Yang
>
> On Mon, Sep 25, 2023 at 3:25 PM Xiangying Meng <xi...@apache.org> wrote:
> >
> > Hi dev,
> > This thread is to start a vote for PIP-302 Add new API
> > readAllExistingMessages for TableView.
> > Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> > PIP: https://github.com/apache/pulsar/pull/21166
> >
> > BR,
> > Xiangying
Re: [VOTE] PIP-302 Add new API readAllExistingMessages for TableView
Posted by Zike Yang <zi...@apache.org>.
Hi, Xiangying
This PIP is a little confusing to me.
This mail title states that we are introducing `readAllExistingMessages`.
But actually, we only introduced `refreshAsync` in the PIP:
https://github.com/apache/pulsar/pull/21166/files#diff-45c655583d6c0c73d87afd3df3fe67f77caadbf1bd691cf8f8211cc89728a1ceR34-R36
And the PR title doesn't seem relevant. “PIP-302 Add alwaysRefresh
Configuration Option for TableView to Read Latest Values”
BR,
Zike Yang
On Mon, Sep 25, 2023 at 3:25 PM Xiangying Meng <xi...@apache.org> wrote:
>
> Hi dev,
> This thread is to start a vote for PIP-302 Add new API
> readAllExistingMessages for TableView.
> Discuss thread: https://lists.apache.org/thread/o085y2314o0fymvx0x8pojmgjwcwn59q
> PIP: https://github.com/apache/pulsar/pull/21166
>
> BR,
> Xiangying