Il giorno gio 10 feb 2022 alle ore 08:39 PengHui Li
<peng...@apache.org> ha scritto:
>
> Enrico,
>
> There are 40 closed PRs
> https://github.com/apache/pulsar/pulls?q=is%3Apr+label%3Arelease%2F2.9.3+is%3Aclosed
> and most of them cherry-picked to branch-2.9, so you mean the 2.9.2

wow, this is incredible, we really should slow down in cherry-picking
stuff to release branches.
But this is off-topic, as this is a VOTE thread.

> contains all of them? or just contain
> https://github.com/apache/pulsar/pull/14192 ?
> I have no objection if we roll out the new RC based on current branch-2.9
> since
> there are some critical fixes for transactions(Of course, I don't think
> there will be any problems moving them to 2.9.3)
> If only for https://github.com/apache/pulsar/pull/14192, I don't think it's
> necessary to roll out the new RC.

I believe it is better to cut a new release candidate out of branch-2.9

Enrico

>
> Thanks,
> Penghui
>
> On Thu, Feb 10, 2022 at 2:28 PM Enrico Olivelli <eolive...@gmail.com> wrote:
>
> > Gao,
> > The patch https://github.com/apache/pulsar/pull/14192 has been merged.
> > I believe that is better to roll out a new RC and have Pulsar 2.9.2
> > released
> >
> >
> > Thanks for the good discussion
> >
> > Enrico
> >
> > Il giorno gio 10 feb 2022 alle ore 05:11 PengHui Li
> > <peng...@apache.org> ha scritto:
> > >
> > > It should not be a blocking issue, as I mentioned before we can work
> > around
> > > and the issue happens for specific conditions
> > > And there are some other ongoing transaction fixes, for transactions
> > using
> > > the latest branch-2.9 is the best option, not 2.9.2 even contain
> > > https://github.com/apache/pulsar/pull/14097
> > >
> > > So I suggest moving forward with the 2.9.2 release and focusing on the
> > > transaction fixes in 2.9.3.
> > >
> > > Regards,
> > > Penghui
> > >
> > > On Thu, Feb 10, 2022 at 11:02 AM Lin Lin <lin...@apache.org> wrote:
> > >
> > > >
> > > > My personal opinion:
> > > >
> > > > If this is a blocking issue, we should tag the issue and raise it in
> > the
> > > > discussion stage, not in the final release stage, which will waste a
> > lot of
> > > > time of the release manager. But I see this issue is already closed.
> > > > https://github.com/apache/pulsar/pull/14097
> > > >
> > > > We can evaluate the repair time of this issue. If it does not take too
> > > > much time, I think it can be merged into 2.9.2. If it is too late, I
> > > > suggest move it to 2.9.3, since there are other serious issues waiting
> > to
> > > > be released, we can run in small steps.
> > > >
> >

Reply via email to