On Thu, May 6, 2021 at 7:22 PM vignesh C <vignes...@gmail.com> wrote: >
Some comments: 1. I don't see any change in pg_dump.c, don't we need to dump this option? 2. + /* Try to connect to the publisher. */ + wrconn = walrcv_connect(sub->conninfo, true, sub->name, &err); + if (!wrconn) + ereport(ERROR, + (errmsg("could not connect to the publisher: %s", err))); Instead of using global wrconn, I think you should use a local variable? -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com