Re: pgsql/src/backend/commands sequence.c

"Vadim B. Mikheev - CVS" <vadim@hub.org> writes:
> Modified files:
>     src/backend/commands: sequence.c

> Log message:
>     Log sequence creation (to initialize magic number on recovery).

This change has at least one serious bug: you access and update the
sequence's buffer after you have done ReleaseBuffer.  Shouldn't the
ReleaseBuffer call be after the critical section?

Also, why'd you replace WriteBuffer with ReleaseBuffer anyway?  Doesn't
the buffer need to get marked dirty here??

            regards, tom lane