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,
--
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 |
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? |