Close this via https://lists.apache.org/thread/vox93tmj33mms026wt52l92h1wffctbk
On Mon, Sep 25, 2023 at 6:34 PM Xiangying Meng <xiangy...@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 <z...@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 <xiangy...@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
