Hi all,

While reviewing another patch, I have noticed that recovery_min_apply_delay
can have a negative value. And the funny part is that we actually attempt
to apply a delay even in this case, per se this condition
recoveryApplyDelay@xlog.c:
        /* nothing to do if no delay configured */
        if (recovery_min_apply_delay == 0)
                return false;
Shouldn't we simply leave if recovery_min_apply_delay is lower 0, and not
only equal to 0?
Regards,
-- 
Michael

Reply via email to