On Thu, 2024-12-19 at 21:23 -0800, Jeff Davis wrote:
> > 0001-0005 - changes to pg_dump/pg_upgrade
>
> Attached is a version 36j...
The testing can use some work here. I noticed that if I take out the
stats entirely, the tests still pass, because pg_upgrade still gets the
same before/after result.
Also, we need some testing of the output and ordering of pg_dump.
Granted, in most cases problems would result in errors during the
reload. But we have those tests for other kinds of objects, so we
should have the tests for stats, too.
I like the description "STATISTICS DATA" because it differentiates from
the extended stats definitions. It might be worth differentiating
between "RELATION STATISTICS DATA" and "ATTRIBUTE STATISTICS DATA" but
I'm not sure if there's value in that.
But how did you determine what to use for the .tag and prefix? In the
output, it uses the form:
Name: STATISTICS DATA <name>; Type: STATISTICS DATA; ...
Should that be:
Name: <name>; Type: STATISTICS DATA; ...
Or:
Data for Name: ...; Name: ...; Type: STATISTICS DATA; ...
Or:
Statistics for Name: ...; Name: ...; Type: STATISTICS DATA; ...
Regards,
Jeff Davis
