| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: GucContext of log_autovacuum |
| Date: | 2007-09-10 19:45:44 |
| Message-ID: | 200709101945.l8AJjiv21775@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
FYI, this has been committed by Tom.
---------------------------------------------------------------------------
ITAGAKI Takahiro wrote:
> The GucContext of log_autovacuum is PGC_BACKEND in the CVS HEAD,
> but should it be PGC_SIGHUP? We cannot modify the variable on-the-fly
> because the parameter is used only by autovacuum worker processes.
> The similar variables, like autovacuum_vacuum_scale_factor, are
> defined as PGC_SIGHUP.
>
>
> Index: src/backend/utils/misc/guc.c
> ===================================================================
> --- src/backend/utils/misc/guc.c (head)
> +++ src/backend/utils/misc/guc.c (working copy)
> @@ -1552,7 +1552,7 @@
> },
>
> {
> - {"log_autovacuum", PGC_BACKEND, LOGGING_WHAT,
> + {"log_autovacuum", PGC_SIGHUP, LOGGING_WHAT,
> gettext_noop("Sets the minimum execution time above which autovacuum actions "
> "will be logged."),
> gettext_noop("Zero prints all actions. The default is -1 (turning this feature off)."),
>
> Regards,
> ---
> ITAGAKI Takahiro
> NTT Open Source Software Center
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Have you searched our list archives?
>
> http://archives.postgresql.org
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Teodor Sigaev | 2007-09-10 20:33:52 | Re: ispell dictionary broken in CVS HEAD ? |
| Previous Message | Florian G. Pflug | 2007-09-10 17:21:49 | Re: Maybe some more low-hanging fruit in the latestCompletedXid patch. |