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. > > > > > >