Hi Magnus, On 3/19/16 8:15 AM, Magnus Hagander wrote:
> I've attached an updated patch, which is rebased on current master and > includes the oid fix. Before doing a thorough review of this patch there are a few points I would like to consider: * I think it's really important to provide the stop time in some fashion when using this new technique. I would prefer a new column to be returned from pg_stop_backup() but I could live with STOP TIME being recorded in the label file. STOP TIME should probably be included in the label file anyway. * It seems like STOP WAL LOCATION should now also be recorded in the label file. Preferably this would used by recovery to determine when the database has reach consistency but that could be a future patch. I'm not very happy with the current method of using pg_control to get this information as it assumes that pg_control is copied last (at least based on the code comments). Thanks, -- -David da...@pgmasters.net -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers