Re: [HACKERS] Optional message to user when terminating/cancellingbackend
От | Alvaro Herrera |
---|---|
Тема | Re: [HACKERS] Optional message to user when terminating/cancellingbackend |
Дата | |
Msg-id | 20180124154548.gmpyvkzlsijren7u@alvherre.pgsql обсуждение исходный текст |
Ответ на | Re: [HACKERS] Optional message to user when terminating/cancelling backend (Daniel Gustafsson <daniel@yesql.se>) |
Ответы |
Re: [HACKERS] Optional message to user when terminating/cancellingbackend
Re: [HACKERS] Optional message to user when terminating/cancelling backend |
Список | pgsql-hackers |
A quick suggestion from a passer-by -- would you put the new code in src/backend/storage/ipc/backend_signal.c instead? Sounds like a better place than utils/misc/; and "signal" is more general in nature than just "cancel". A bunch of stuff from utils/adt/misc.c (???) can migrate to the new file -- everything from line 201 to 362 at least, that is: pg_signal_backend pg_cancel_backend pg_terminate_backend pg_reload_conf pg_rotate_logfile Maybe have two patches, 0001 creates the files moving the contents over, then 0002 adds your new stuff on top. /me wonders if there's anything that would suggest to make this extensive to processes other than backends ... -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
В списке pgsql-hackers по дате отправления: