Inconsistent usage of BACKEND_* symbols

Поиск
Список
Период
Сортировка
От Kyotaro Horiguchi
Тема Inconsistent usage of BACKEND_* symbols
Дата
Msg-id 20190930.163959.100970012.horikyota.ntt@gmail.com
обсуждение исходный текст
Ответы Re: Inconsistent usage of BACKEND_* symbols  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers
Hello.

While I looked around shutdown sequence, pmdie() uses
"BACKEND_TYPE_AUTOVAC | BACKEND_TYPE_BGWORKER" for sending signal
and PostmasterStateMachine counts them using
BACKEND_TYPE_WORKER. It is the only usage of the combined one. It
seems to me just a leftover of da07a1e856.

regards.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index eb9e0221f8..a071c3de87 100644
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -2740,8 +2740,8 @@ pmdie(SIGNAL_ARGS)
             {
                 /* autovac workers are told to shut down immediately */
                 /* and bgworkers too; does this need tweaking? */
-                SignalSomeChildren(SIGTERM,
-                                   BACKEND_TYPE_AUTOVAC | BACKEND_TYPE_BGWORKER);
+                SignalSomeChildren(SIGTERM, BACKEND_TYPE_WORKER);
+
                 /* and the autovac launcher too */
                 if (AutoVacPID != 0)
                     signal_child(AutoVacPID, SIGTERM);
@@ -2821,8 +2821,7 @@ pmdie(SIGNAL_ARGS)
                         (errmsg("aborting any active transactions")));
                 /* shut down all backends and workers */
                 SignalSomeChildren(SIGTERM,
-                                   BACKEND_TYPE_NORMAL | BACKEND_TYPE_AUTOVAC |
-                                   BACKEND_TYPE_BGWORKER);
+                                   BACKEND_TYPE_NORMAL | BACKEND_TYPE_WORKER);
                 /* and the autovac launcher too */
                 if (AutoVacPID != 0)
                     signal_child(AutoVacPID, SIGTERM);

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Batch insert in CTAS/MatView code
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Don't codegen deform code for virtual tuples in expr eval forscan fetch