Hi Chao,
Thanks for the v5 patch (all the points that were earlier raised were addressed). Both patches apply with git am, the build is clean with no new warnings.
I also checked that advancing the origin doesn't leak into the rest of
subscription.sql: no later test sets a skip LSN with a valid location, and the
new block leaves subskiplsn reset to NONE, so nothing downstream depends on the origin position.
No further comments from me, v5 looks ready for a committer. You can create a commitfest entry for this.
Regards,
Surya Poondla
