diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-05-12 19:13:54 -0400 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-05-12 19:13:54 -0400 |
| commit | 6e9723cde5c672bb55ed803fc42720d0a46ce03c (patch) | |
| tree | ee5e30e3954f5bfb2c7c460467463476c2fbda25 /src/backend/replication/logical/launcher.c | |
| parent | 4bf0bce161097869be5a56706b31388ba15e0113 (diff) | |
| download | postgresql-6e9723cde5c672bb55ed803fc42720d0a46ce03c.tar.gz postgresql-6e9723cde5c672bb55ed803fc42720d0a46ce03c.zip | |
Rename the logical replication global "wrconn"
The worker.c global wrconn is only meant to be used by logical apply/
tablesync workers, but there are other variables with the same name. To
reduce future confusion rename the global from "wrconn" to
"LogRepWorkerWalRcvConn".
While this is just cosmetic, it seems better to backpatch it all the way
back to 10 where this code appeared, to avoid future backpatching
issues.
Author: Peter Smith <smithpb2250@gmail.com>
Discussion: https://postgr.es/m/CAHut+Pu7Jv9L2BOEx_Z0UtJxfDevQSAUW2mJqWU+CtmDrEZVAg@mail.gmail.com
Diffstat (limited to 'src/backend/replication/logical/launcher.c')
| -rw-r--r-- | src/backend/replication/logical/launcher.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/replication/logical/launcher.c b/src/backend/replication/logical/launcher.c index 186057bd932..d8ff8664fb6 100644 --- a/src/backend/replication/logical/launcher.c +++ b/src/backend/replication/logical/launcher.c @@ -716,8 +716,8 @@ static void logicalrep_worker_onexit(int code, Datum arg) { /* Disconnect gracefully from the remote side. */ - if (wrconn) - walrcv_disconnect(wrconn); + if (LogRepWorkerWalRcvConn) + walrcv_disconnect(LogRepWorkerWalRcvConn); logicalrep_worker_detach(); |