On Wed, Aug 13, 2014 at 4:49 PM, Michael Paquier <michael.paqu...@gmail.com> wrote: > Yes indeed. As XLogBeginInsert() is already part of xlogconstruct.c, > it is not weird. This approach has the merit to make XLogRecData > completely bundled with the insertion and construction of the WAL > records. Then for the name xloginsert.c is fine for me too. At the same time, renaming XLogInsert to XLogCompleteInsert or XLogFinishInsert would be nice to make the difference with XLogBeginInsert. This could include XLogCancel renamed tos XLogCancelInsert.
Appending the prefix XLogInsert* to those functions would make things more consistent as well. But feel free to discard those ideas if you do not like that. Regards, -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
