| From: | "Jelte Fennema-Nio" <postgres(at)jeltef(dot)nl> |
|---|---|
| To: | "Kirill Reshke" <reshkekirill(at)gmail(dot)com>, "Jelte Fennema-Nio" <me(at)jeltef(dot)nl> |
| Cc: | "PostgreSQL Hackers" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "Dave Cramer" <davecramer(at)gmail(dot)com>, "Jacob Champion" <jacob(dot)champion(at)enterprisedb(dot)com>, "Heikki Linnakangas" <hlinnaka(at)iki(dot)fi> |
| Subject: | Re: Add GoAway protocol message for graceful but fast server shutdown/switchover |
| Date: | 2025-10-24 11:54:37 |
| Message-ID: | DDQJ6HE66UVK.3E1XVYZIVCID8@jeltef.nl |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri Oct 24, 2025 at 7:04 AM CEST, Kirill Reshke wrote:
> On Thu, 23 Oct 2025 at 18:05, Jelte Fennema-Nio <me(at)jeltef(dot)nl> wrote:
> Im +1 on this idea. This is something I wanted back in 2020, when
> implementing the 'online restart' feature for odyssey[0], but never
> bothered to create a thread.
Yeah, to be clear: A big goal of this is definitely to be used by
poolers/proxies/middleware. Those systems will often be more frequently
restarted than the actual database servers, so being able to do that
quickly without disrupting active connections is much more important
there than with plain PostgreSQL servers.
> About patches:
Thanks for the review. Attached is a new patchset. I think I addressed
all of your comments (I almost fully rewrote the docs). I also fixed
two other issues that I found:
- updating docs for 3.3 in more places
- handling the GoAway message in more code paths on the client side
| Attachment | Content-Type | Size |
|---|---|---|
| nocfbot.v2-0003-Add-pytest-based-tests-for-GoAway-message.patch | text/x-patch | 6.9 KB |
| v2-0001-Bump-protocol-version-to-3.3.patch | text/x-patch | 7.4 KB |
| v2-0002-Add-GoAway-protocol-message-for-graceful-but-fast.patch | text/x-patch | 19.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Joel Jacobson | 2025-10-24 12:12:01 | Re: LISTEN/NOTIFY bug: VACUUM sets frozenxid past a xid in async queue |
| Previous Message | Joe Conway | 2025-10-24 11:49:15 | Re: contrib/sepgsql regression tests have been broken for months |