On Sat, Sep 22, 2018 at 9:55 AM Amit Kapila <amit.kapil...@gmail.com> wrote:
> On Fri, Sep 21, 2018 at 11:06 PM Alexander Korotkov > <a.korot...@postgrespro.ru> wrote: > > While investigating ProcArrayGroupClearXid() code I wonder why do we > have this loop instead of plain pg_atomic_exchange_u32() call? > > We can use pg_atomic_exchange_u32 instead of a loop. In fact, clog > code uses pg_atomic_exchange_u32 for the same purpose. I think it is > better to make the code consistent at both places. Feel free to > change it, otherwise, I can take care of it in a few days time. > Thank you for feedback. Pushed. ------ Alexander Korotkov Postgres Professional: http://www.postgrespro.com The Russian Postgres Company
