Re: pgsql: Allow background workers to bypass datallowconn

From: Teodor Sigaev <teodor(at)sigaev(dot)ru>
To: Magnus Hagander <magnus(at)hagander(dot)net>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Allow background workers to bypass datallowconn
Date: 2018-04-05 17:11:44
Message-ID: 07be884c-6415-4868-4253-15a97d60071f@sigaev.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Something goes wrong

https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=crake&dt=2018-04-05
17%3A07%3A19

Apr 05 13:09:08
/home/bf/bfr/root/HEAD/pgsql.build/../pgsql/src/test/modules/worker_spi/worker_spi.c:
In function ‘worker_spi_main’:
Apr 05 13:09:08
/home/bf/bfr/root/HEAD/pgsql.build/../pgsql/src/test/modules/worker_spi/worker_spi.c:182:2:
error: too few arguments to function ‘BackgroundWorkerInitializeConnection’
Apr 05 13:09:08 BackgroundWorkerInitializeConnection("postgres", NULL);
Apr 05 13:09:08 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Apr 05 13:09:08 In file included from
/home/bf/bfr/root/HEAD/pgsql.build/../pgsql/src/test/modules/worker_spi/worker_spi.c:27:0:
Apr 05 13:09:08
/home/bf/bfr/root/HEAD/pgsql.build/../pgsql/src/include/postmaster/bgworker.h:143:13:
note: declared here
Apr 05 13:09:08 extern void BackgroundWorkerInitializeConnection(const char
*dbname, const char *username, uint32 flags);
Apr 05 13:09:08 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Apr 05 13:09:08 <builtin>: recipe for target 'worker_spi.o' failed

Magnus Hagander wrote:
> Allow background workers to bypass datallowconn
>
> THis adds a "flags" field to the BackgroundWorkerInitializeConnection()
> and BackgroundWorkerInitializeConnectionByOid(). For now only one flag,
> BGWORKER_BYPASS_ALLOWCONN, is defined, which allows the worker to ignore
> datallowconn.
>
> Branch
> ------
> master
>
> Details
> -------
> https://git.postgresql.org/pg/commitdiff/eed1ce72e1593d3e8b7461d7744808d4d6bf402b
>
> Modified Files
> --------------
> contrib/pg_prewarm/autoprewarm.c | 2 +-
> src/backend/access/transam/parallel.c | 3 ++-
> src/backend/bootstrap/bootstrap.c | 2 +-
> src/backend/postmaster/autovacuum.c | 4 ++--
> src/backend/postmaster/postmaster.c | 8 ++++----
> src/backend/replication/logical/launcher.c | 2 +-
> src/backend/replication/logical/worker.c | 3 ++-
> src/backend/tcop/postgres.c | 2 +-
> src/backend/utils/init/postinit.c | 10 +++++-----
> src/include/miscadmin.h | 2 +-
> src/include/postmaster/bgworker.h | 7 +++++--
> 11 files changed, 25 insertions(+), 20 deletions(-)
>

--
Teodor Sigaev E-mail: teodor(at)sigaev(dot)ru
WWW: http://www.sigaev.ru/

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Magnus Hagander 2018-04-05 17:14:59 Re: pgsql: Allow background workers to bypass datallowconn
Previous Message Teodor Sigaev 2018-04-05 17:06:24 pgsql: Remove tsearch test contained russian characters, missed in