You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by Shen Liu <li...@126.com> on 2023/11/15 08:33:06 UTC
[VOTE] PIP-317: Add `bookkeeperDeleted` field to show whether a ledger is deleted from the Bookie while using tiered storage
Hi community,
This thread is to start a vote for PIP-317: Add `bookkeeperDeleted` field to show whether a ledger is deleted from the Bookie while using tiered storage.
PIP: https://github.com/apache/pulsar/pull/21521
Discussion thread: https://lists.apache.org/thread/9kjgg06oj384myxqls73dwn840thtrrc
Releted PR/issue: https://github.com/apache/pulsar/pull/20833
--
Best Regards,
Shen Liu
Re: [VOTE] PIP-317: Add `bookkeeperDeleted` field to show whether a ledger is deleted from the Bookie while using tiered storage
Posted by Hang Chen <ch...@apache.org>.
+1 (binding)
Regards,
Hang
PengHui Li <pe...@apache.org> 于2023年11月27日周一 09:54写道：
>
> +1 (binding)
>
> And please also update the `# Backward & Forward Compatibility` section
> as Enrico mentioned.
>
> Regards,
> Penghui
>
> On Thu, Nov 16, 2023 at 12:47 PM Enrico Olivelli <eo...@gmail.com>
> wrote:
>
> > +1
> >
> > It will be important to add tests about rolling back to the previous
> > version of Pulsar and/or clusters with mixed versions of the brokers
> > (during rolling upgrades).
> > The old version should not complain about the new field added.
> > Maybe you can add a note in the PIP
> >
> > Enrico
> >
> > Il Mer 15 Nov 2023, 00:35 Shen Liu <li...@126.com> ha scritto:
> >
> > > Hi community,
> > >
> > > This thread is to start a vote for PIP-317: Add `bookkeeperDeleted` field
> > > to show whether a ledger is deleted from the Bookie while using tiered
> > > storage.
> > >
> > >
> > >
> > > PIP: https://github.com/apache/pulsar/pull/21521
> > >
> > > Discussion thread:
> > > https://lists.apache.org/thread/9kjgg06oj384myxqls73dwn840thtrrc
> > >
> > > Releted PR/issue: https://github.com/apache/pulsar/pull/20833
> > >
> > >
> > >
> > >
> > > --
> > >
> > > Best Regards,
> > > Shen Liu
> >
Re: [VOTE] PIP-317: Add `bookkeeperDeleted` field to show whether a ledger is deleted from the Bookie while using tiered storage
Posted by PengHui Li <pe...@apache.org>.
+1 (binding)
And please also update the `# Backward & Forward Compatibility` section
as Enrico mentioned.
Regards,
Penghui
On Thu, Nov 16, 2023 at 12:47 PM Enrico Olivelli <eo...@gmail.com>
wrote:
> +1
>
> It will be important to add tests about rolling back to the previous
> version of Pulsar and/or clusters with mixed versions of the brokers
> (during rolling upgrades).
> The old version should not complain about the new field added.
> Maybe you can add a note in the PIP
>
> Enrico
>
> Il Mer 15 Nov 2023, 00:35 Shen Liu <li...@126.com> ha scritto:
>
> > Hi community,
> >
> > This thread is to start a vote for PIP-317: Add `bookkeeperDeleted` field
> > to show whether a ledger is deleted from the Bookie while using tiered
> > storage.
> >
> >
> >
> > PIP: https://github.com/apache/pulsar/pull/21521
> >
> > Discussion thread:
> > https://lists.apache.org/thread/9kjgg06oj384myxqls73dwn840thtrrc
> >
> > Releted PR/issue: https://github.com/apache/pulsar/pull/20833
> >
> >
> >
> >
> > --
> >
> > Best Regards,
> > Shen Liu
>
Re: [VOTE] PIP-317: Add `bookkeeperDeleted` field to show whether a ledger is deleted from the Bookie while using tiered storage
Posted by Enrico Olivelli <eo...@gmail.com>.
+1
It will be important to add tests about rolling back to the previous
version of Pulsar and/or clusters with mixed versions of the brokers
(during rolling upgrades).
The old version should not complain about the new field added.
Maybe you can add a note in the PIP
Enrico
Il Mer 15 Nov 2023, 00:35 Shen Liu <li...@126.com> ha scritto:
> Hi community,
>
> This thread is to start a vote for PIP-317: Add `bookkeeperDeleted` field
> to show whether a ledger is deleted from the Bookie while using tiered
> storage.
>
>
>
> PIP: https://github.com/apache/pulsar/pull/21521
>
> Discussion thread:
> https://lists.apache.org/thread/9kjgg06oj384myxqls73dwn840thtrrc
>
> Releted PR/issue: https://github.com/apache/pulsar/pull/20833
>
>
>
>
> --
>
> Best Regards,
> Shen Liu