On Tue, Mar 29, 2016 at 3:17 AM, Michael Paquier <michael.paqu...@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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
