Re: recoveryStopsAfter is not usable when recovery_target_inclusive is false

2025-07-24 Thread Shlok Kyal
On Thu, 24 Jul 2025 at 16:49, Hayato Kuroda (Fujitsu) wrote: > > Dear Michael, > > Sorry for the late reply. > > > So, what you are doing here is changing the behavior of the check in > > recoveryStopsAfter(), with the addition of one exit path based on > > EndRecPtr if recovery_target_inclusive i

RE: recoveryStopsAfter is not usable when recovery_target_inclusive is false

2025-07-24 Thread Hayato Kuroda (Fujitsu)
Dear Michael, Sorry for the late reply. > So, what you are doing here is changing the behavior of the check in > recoveryStopsAfter(), with the addition of one exit path based on > EndRecPtr if recovery_target_inclusive is false, to leave a bit > earlier in case if we don't have a follow-up recor

Re: recoveryStopsAfter is not usable when recovery_target_inclusive is false

2025-07-23 Thread Michael Paquier
On Wed, Jul 23, 2025 at 05:04:37AM +, Hayato Kuroda (Fujitsu) wrote: > While working on [1] I found the point. When recovery_target_lsn is specified > and > recovery_target_inclusive is false, recoveryStopsAfter() cannot return true. > > /* Check if the target LSN has been reached */ >

recoveryStopsAfter is not usable when recovery_target_inclusive is false

2025-07-22 Thread Hayato Kuroda (Fujitsu)
Hi hackers, While working on [1] I found the point. When recovery_target_lsn is specified and recovery_target_inclusive is false, recoveryStopsAfter() cannot return true. ``` /* Check if the target LSN has been reached */ if (recoveryTarget == RECOVERY_TARGET_LSN &&