On Tue, Mar 29, 2016 at 3:17 AM, Michael Paquier
<michael.paquier@gmail.com> wrote:
> OK, so I am switching this patch as "Ready for committer", for 0001.
> It is in better shape now.
Well... I have a few questions yet.
The new argument to SyncRepWaitForLSN is called "bool commit", but
RecordTransactionAbortPrepared passes true. Either it should be
passing false, or the parameter is misnamed or at the least in need of
a better comment.
I don't understand why this patch is touching the abort paths at all.
XactLogAbortRecord sets XACT_COMPLETION_SYNC_APPLY_FEEDBACK, and
xact_redo_abort honors it. But surely it makes no sense to wait for
an abort to become visible.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company