From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, "Bossart, Nathan" <bossartn(at)amazon(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Michael Paquier <michael(at)paquier(dot)xyz> |
Subject: | Re: New vacuum option to do only freezing |
Date: | 2019-05-17 20:55:49 |
Message-ID: | 20190517205549.vsr6jl232uxwxmin@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2019-05-09 14:14:20 +0900, Masahiko Sawada wrote:
> I agree that this item is neither critical and bug. But this is an
> (my) oversight and is a small patch and I think there is no specific
> reason why we don't dare to include this in 12. So if this patch could
> get reviewed enough I think we can have it in 12. Since the previous
> patch conflicts with current HEAD I've attached the rebased version
> patch.
Robert, this indeed looks near trivial. What do you think?
> diff --git a/doc/src/sgml/ref/create_table.sgml b/doc/src/sgml/ref/create_table.sgml
> index 44a61ef..1e1b0e8 100644
> --- a/doc/src/sgml/ref/create_table.sgml
> +++ b/doc/src/sgml/ref/create_table.sgml
> @@ -1406,7 +1406,7 @@ WITH ( MODULUS <replaceable class="parameter">numeric_literal</replaceable>, REM
> </varlistentry>
>
> <varlistentry id="reloption-vacuum-index-cleanup" xreflabel="vacuum_index_cleanup">
> - <term><literal>vacuum_index_cleanup</literal> (<type>boolean</type>)
> + <term><literal>vacuum_index_cleanup</literal>, <literal>toast.vacuum_index_cleanup</literal> (<type>boolean</type>)
> <indexterm>
> <primary><varname>vacuum_index_cleanup</varname> storage parameter</primary>
> </indexterm>
> diff --git a/src/backend/access/common/reloptions.c b/src/backend/access/common/reloptions.c
> index cfbabb5..022b3a0 100644
> --- a/src/backend/access/common/reloptions.c
> +++ b/src/backend/access/common/reloptions.c
> @@ -147,7 +147,7 @@ static relopt_bool boolRelOpts[] =
> {
> "vacuum_index_cleanup",
> "Enables index vacuuming and index cleanup",
> - RELOPT_KIND_HEAP,
> + RELOPT_KIND_HEAP | RELOPT_KIND_TOAST,
> ShareUpdateExclusiveLock
> },
> true
> diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c
> index e4c03de..2379b3d 100644
> --- a/src/bin/psql/tab-complete.c
> +++ b/src/bin/psql/tab-complete.c
> @@ -1056,6 +1056,7 @@ static const char *const table_storage_parameters[] = {
> "toast.autovacuum_vacuum_scale_factor",
> "toast.autovacuum_vacuum_threshold",
> "toast.log_autovacuum_min_duration",
> + "toast.vacuum_index_clenaup",
> "toast.vacuum_truncate",
> "toast_tuple_target",
> "user_catalog_table",
typo.
Sawada-san, it'd be good if you could add at least some minimal tests in
the style of the no_index_cleanup test in vacuum.sql.
- Andres
From | Date | Subject | |
---|---|---|---|
Next Message | Ashwin Agrawal | 2019-05-17 21:12:12 | Re: Create TOAST table only if AM needs |
Previous Message | Andres Freund | 2019-05-17 20:51:54 | Re: Create TOAST table only if AM needs |