Re: [VOTE] PIP-302 Add new API readAllExistingMessages for TableView

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

Re: [VOTE] PIP-302 Add new API readAllExistingMessages for TableView

2023-09-25 Thread Xiangying Meng
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 m

Re: [VOTE] PIP-302 Add new API readAllExistingMessages for TableView

2023-09-25 Thread Zike Yang
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-45c655583d6c0c73d87afd3df3fe67f77caadbf1bd691cf8f8211cc89

[VOTE] PIP-302 Add new API readAllExistingMessages for TableView

2023-09-25 Thread Xiangying Meng
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