Simplifying the interface of UpdateMinRecoveryPoint - Mailing list pgsql-hackers

From Michael Paquier
Subject Simplifying the interface of UpdateMinRecoveryPoint
Date
Msg-id CAB7nPqQjgNLZTH5KTp32QK19k7LyiW6OUZBDBSoJios=nxPk_A@mail.gmail.com
Whole thread Raw
Responses Re: Simplifying the interface of UpdateMinRecoveryPoint  (Amit Kapila <amit.kapila16@gmail.com>)
Re: Simplifying the interface of UpdateMinRecoveryPoint  (Simon Riggs <simon@2ndquadrant.com>)
List pgsql-hackers
Hi all,

As of now UpdateMinRecoveryPoint() is using two arguments:
- lsn, to check if the minimum recovery point should be updated to that
- force, a boolean flag to decide if the update should be enforced or not.
However those two arguments are correlated. If lsn is
InvalidXlogRecPtr, the minimum recovery point update will be enforced.
Hence why not simplifying its interface and remove the force flag? See
attached.

Thanks,
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: RecoveryTargetTLI dead variable in XLogCtlData
Next
From: Michael Paquier
Date:
Subject: Re: dumping database privileges broken in 9.6