Re: ANALYZE ONLY

From: Michael Harris <harmic(at)gmail(dot)com>
To: torikoshia <torikoshia(at)oss(dot)nttdata(dot)com>
Cc: David Rowley <dgrowleyml(at)gmail(dot)com>, Melih Mutlu <m(dot)melihmutlu(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, postgres(at)jeltef(dot)nl, ilya(dot)evdokimov(at)tantorlabs(dot)com
Subject: Re: ANALYZE ONLY
Date: 2024-09-11 07:47:09
Message-ID: CADofcAXaB7Ouo2i9jpK0HesHQFBa=KhCx0uKTwEc2MAjcwVJUQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Thanks for the feedback.

On Tue, 10 Sept 2024 at 22:03, torikoshia <torikoshia(at)oss(dot)nttdata(dot)com> wrote:
> Regarding the addition of partition descendant tables, should we also
> update the below comment on expand_vacuum_rel? Currently it refers only
> partitions:
>
> | * Given a VacuumRelation, fill in the table OID if it wasn't
> specified,
> | * and optionally add VacuumRelations for partitions of the table.
>

Well spotted! I have updated the comment to read:

* Given a VacuumRelation, fill in the table OID if it wasn't specified,
* and optionally add VacuumRelations for partitions or descendant tables
* of the table.

Cheers
Mike.

Attachment Content-Type Size
v5-0001-Implementation-of-the-ONLY-feature.patch application/octet-stream 16.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2024-09-11 07:51:18 Re: json_query conditional wrapper bug
Previous Message Michael Paquier 2024-09-11 07:41:49 Re: Separate HEAP WAL replay logic into its own file