diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-03-08 11:25:26 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-03-08 11:25:26 -0500 |
| commit | 4e0c743c18bf5435a4850510c5c74b3521c3e1e5 (patch) | |
| tree | 548ba5c052c971a450f9f0bc5db10521b9946a69 /src/backend/utils/init/postinit.c | |
| parent | f9f8784c8b3050aaec52da88a6e41b3e3f576b96 (diff) | |
| download | postgresql-4e0c743c18bf5435a4850510c5c74b3521c3e1e5.tar.gz postgresql-4e0c743c18bf5435a4850510c5c74b3521c3e1e5.zip | |
Fix cross-checking of ReservedBackends/max_wal_senders/MaxConnections.
We were independently checking ReservedBackends < MaxConnections and
max_wal_senders < MaxConnections, but because walsenders aren't allowed
to use superuser-reserved connections, that's really the wrong thing.
Correct behavior is to insist on ReservedBackends + max_wal_senders being
less than MaxConnections. Fix the code and associated documentation.
This has been wrong for a long time, but since the situation probably
hardly ever arises in the field (especially pre-v10, when the default
for max_wal_senders was zero), no back-patch.
Discussion: https://postgr.es/m/28271.1520195491@sss.pgh.pa.us
Diffstat (limited to 'src/backend/utils/init/postinit.c')
| -rw-r--r-- | src/backend/utils/init/postinit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c index 484628987f4..d8f45b3c43f 100644 --- a/src/backend/utils/init/postinit.c +++ b/src/backend/utils/init/postinit.c @@ -778,7 +778,7 @@ InitPostgres(const char *in_dbname, Oid dboid, const char *username, } /* - * The last few connections slots are reserved for superusers. Although + * The last few connection slots are reserved for superusers. Although * replication connections currently require superuser privileges, we * don't allow them to consume the reserved slots, which are intended for * interactive use. |