On Wed, Aug 13, 2014 at 4:49 PM, Michael Paquier
<michael.paquier@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