From: Heikki Linnakangas Date: Thu, 1 Aug 2024 20:23:47 +0000 (+0300) Subject: Fix outdated comment; all running bgworkers are in BackendList X-Git-Tag: REL_18_BETA1~2224 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=ef4c35b4166e3c99558eba1bc324b2fde61d6ed5;p=thirdparty%2Fpostgresql.git Fix outdated comment; all running bgworkers are in BackendList Before commit 8a02b3d732, only bgworkers that connected to a database had an entry in the Backendlist. Commit 8a02b3d732 changed that, but forgot to update this comment. Discussion: https://www.postgresql.org/message-id/835232c0-a5f7-4f20-b95b-5b56ba57d741@iki.fi --- diff --git a/src/include/postmaster/bgworker_internals.h b/src/include/postmaster/bgworker_internals.h index 9106a0ef3f0..61ba54117a8 100644 --- a/src/include/postmaster/bgworker_internals.h +++ b/src/include/postmaster/bgworker_internals.h @@ -26,14 +26,14 @@ /* * List of background workers, private to postmaster. * - * A worker that requests a database connection during registration will have - * rw_backend set, and will be present in BackendList. Note: do not rely on - * rw_backend being non-NULL for shmem-connected workers! + * All workers that are currently running will have rw_backend set, and will + * be present in BackendList. */ typedef struct RegisteredBgWorker { BackgroundWorker rw_worker; /* its registry entry */ - struct bkend *rw_backend; /* its BackendList entry, or NULL */ + struct bkend *rw_backend; /* its BackendList entry, or NULL if not + * running */ pid_t rw_pid; /* 0 if not running */ int rw_child_slot; TimestampTz rw_crashed_at; /* if not 0, time it last crashed */