On Wed, Nov 23, 2022 at 12:00 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Tue, Nov 22, 2022 at 10:33 PM Maxim Orlov <orlo...@gmail.com> wrote: > >> > >> > >> Regarding the tests, the patch includes a new scenario to > >> reproduce this issue. However, since the issue can be reproduced also > >> by the existing scenario (with low probability, though), I'm not sure > >> it's worth adding the new scenario. > > > > AFAICS, the test added doesn't 100% reproduce this issue, so, maybe, it > > does not worth it. But, I do not have a strong opinion here. > > Let's add tests in a separate commit and let the actual committer to decide > > what to do, should we? > > > > +1 to not have a test for this as the scenario can already be tested > by the existing set of tests.
Agreed not to have a test case for this. I've attached an updated patch. I've confirmed this patch works for all supported branches. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
0001-Reset-InitialRunningXacts-array-when-freeing-SnapBui.patch
Description: Binary data