| From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | "Imseih (AWS), Sami" <simseih(at)amazon(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: allow changing autovacuum_max_workers without restarting |
| Date: | 2024-07-19 19:24:39 |
| Message-ID: | Zpq9dx37tdUkV42U@nathan |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Jul 08, 2024 at 02:29:16PM -0500, Nathan Bossart wrote:
> One thing that still bugs me is that there is no feedback sent to the user
> when autovacuum_max_workers is set higher than autovacuum_worker_slots. I
> think we should at least emit a WARNING, perhaps from the autovacuum
> launcher, i.e., once when the launcher starts and then again as needed via
> HandleAutoVacLauncherInterrupts(). Or we could fail to start in
> PostmasterMain() and then ignore later misconfigurations via a GUC check
> hook. I'm not too thrilled about adding more GUC check hooks that depend
> on the value of other GUCs, but I do like the idea of failing instead of
> silently proceeding with a different value than the user configured. Any
> thoughts?
From recent discussions, it sounds like there isn't much appetite for GUC
check hooks that depend on the values of other GUCs. Here is a new version
of the patch that adds the WARNING described above.
--
nathan
| Attachment | Content-Type | Size |
|---|---|---|
| v8-0001-allow-changing-autovacuum_max_workers-without-res.patch | text/plain | 19.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David E. Wheeler | 2024-07-19 19:27:49 | DSO Terms Galore |
| Previous Message | Tom Lane | 2024-07-19 19:22:57 | Re: Wrong results with grouping sets |