Re: Allow pg_dump --statistics-only to dump foreign table statistics?

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: Corey Huinker <corey(dot)huinker(at)gmail(dot)com>, Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, pgsql(at)j-davis(dot)com
Subject: Re: Allow pg_dump --statistics-only to dump foreign table statistics?
Date: 2025-06-17 03:39:16
Message-ID: 60a37349-f5df-42e4-aa0b-b734a452597c@oss.nttdata.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2025/06/17 7:44, Corey Huinker wrote:
> The proposed patch [0] adds RELKIND_FOREIGN_TABLE to this list.  That
> appears to be the only missing relation kind that ANALYZE handles.
>
> [0] https://postgr.es/m/attachment/177608/v1-0001-pg_dump-Allow-pg_dump-to-dump-the-statistics-for-.patch <https://postgr.es/m/attachment/177608/v1-0001-pg_dump-Allow-pg_dump-to-dump-the-statistics-for-.patch>
>
>
> Thanks for pointing it out, a little distracted today.
>
> +1 for the patch.

Thanks for reviewing the patch! I found a few more descriptions that
needed updates, so I've revised the patch accordingly. The updated
version is attached. Barring any objections, I plan to commit it.

</para>
<para>
The data section contains actual table data, large-object
- contents, statistics for tables and materialized views and
- sequence values.
+ contents, sequence values, and statistics for tables,
+ materialized views, and foriegn tables.
Post-data items include definitions of indexes, triggers, rules,
statistics for indexes, and constraints other than validated check
constraints.

Although not directly related to foreign table statistics, I considered
clarifying that the "data" section only includes statistics *if*
--statistics is specified. The current wording might suggest that
statistics are included by default, which isn't accurate. However,
since the default behavior is still under discussion at [1],
I've left that part unchanged for now.

Regards,

[1] https://www.postgresql.org/message-id/CADkLM=fXSiX4GQ7F6__z+ofGpp0QwwytGP-GNeVQyMhc29bbwQ@mail.gmail.com

--
Fujii Masao
NTT DATA Japan Corporation

Attachment Content-Type Size
v2-0001-pg_dump-Allow-pg_dump-to-dump-the-statistics-for-.patch text/plain 3.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Hayato Kuroda (Fujitsu) 2025-06-17 03:47:00 RE: Suggestion to add --continue-client-on-abort option to pgbench
Previous Message Fujii Masao 2025-06-17 03:34:28 Re: Allow pg_dump --statistics-only to dump foreign table statistics?