The following review has been posted through the commitfest application:
make installcheck-world:  not tested
Implements feature:       not tested
Spec compliant:           not tested
Documentation:            not tested

The logic to start WAL receiver early should not be coupled with 
recovery_min_apply_delay GUC.  WAL receiver's delayed start affects replication 
in general, even when the GUC is not set.

A better fix would be to start WAL receiver in the main replay loop, as soon as 
consistent state has been reached.

As noted during previous reviews, scanning all WAL just to determine streaming 
start point seems slow.  A faster solution seems desirable.

The new status of this patch is: Waiting on Author

Reply via email to