Alexander Korotkov writes:
> Sorry for delayed response. Yes, my fix for oversight contain
> oversight itself. I confirm the fix you've committed is correct.
> Thank you!
Thanks for checking!
regards, tom lane
On Sun, Feb 9, 2020 at 8:05 PM Tom Lane wrote:
>
> I wrote:
> > Alexander Korotkov writes:
> >> Fix page modification outside of critical section in GIN
>
> > I happened to notice, while reviewing stuff for the release notes,
> > that this patch does not do what the commit message says. The
> >
I wrote:
> Alexander Korotkov writes:
>> Fix page modification outside of critical section in GIN
> I happened to notice, while reviewing stuff for the release notes,
> that this patch does not do what the commit message says. The
> previous code modified the pd_prune_xid of the "dBuffer" page,
Alexander Korotkov writes:
> Fix page modification outside of critical section in GIN
> By oversight 52ac6cd2d0 makes ginDeletePage() sets pd_prune_xid of page to be
> deleted before entering the critical section. It appears that only versions
> 11
> and later were affected by this oversight.
I
Fix page modification outside of critical section in GIN
By oversight 52ac6cd2d0 makes ginDeletePage() sets pd_prune_xid of page to be
deleted before entering the critical section. It appears that only versions 11
and later were affected by this oversight.
Backpatch-through: 11
Branch
--
ma
Fix page modification outside of critical section in GIN
By oversight 52ac6cd2d0 makes ginDeletePage() sets pd_prune_xid of page to be
deleted before entering the critical section. It appears that only versions 11
and later were affected by this oversight.
Backpatch-through: 11
Branch
--
RE
Fix page modification outside of critical section in GIN
By oversight 52ac6cd2d0 makes ginDeletePage() sets pd_prune_xid of page to be
deleted before entering the critical section. It appears that only versions 11
and later were affected by this oversight.
Backpatch-through: 11
Branch
--
RE
