From: | Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> |
---|---|
To: | Tatsuro Yamada <tatsuro(dot)yamada(dot)tf(at)nttcom(dot)co(dot)jp>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: list of extended statistics on psql |
Date: | 2020-11-15 18:22:59 |
Message-ID: | 1123e0bb-a95e-dd4f-eff5-97a668b48077@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Thanks,
It's better to always post the whole patch series, so that cfbot can
test it properly. Sending just 0003 separately kind breaks that.
Also, 0003 seems to only tweak the .sql file, not the expected output,
and there actually seems to be two places that mistakenly use \dx (so
listing extensions) instead of \dX. I've fixed both issues in the
attached patches.
However, I think the 0002 tests are better/sufficient - I prefer to keep
it compact, not interleaving with the tests testing various other stuff.
So I don't intend to commit 0003, unless there's something that I don't
see for some reason.
The one remaining thing I'm not sure about is naming of the columns with
size of statistics - N_size, D_size and M_size does not seem very clear.
Any clearer naming will however make the tables wider, though :-/
regards
--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
0001-Add-dX-command-on-psql-20201115.patch | text/x-patch | 7.8 KB |
0002-Add-regression-test-of-dX-to-stats_ext.sql-20201115.patch | text/x-patch | 9.3 KB |
0003-Add-regression-test-of-dX-to-stats_ext.sql--20201115.patch | text/x-patch | 27.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Jürgen Purtz | 2020-11-15 18:45:35 | Re: Additional Chapter for Tutorial |
Previous Message | Tom Lane | 2020-11-15 17:16:56 | Re: Move OpenSSL random under USE_OPENSSL_RANDOM |