From: Tomas Vondra <tomas.vondra@enterprisedb.com>
To: Tatsuro Yamada <tatsuro.yamada.tf@nttcom.co.jp>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Tomas Vondra <tomas.vondra@2ndquadrant.com>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Pavel Stehule <pavel.stehule@gmail.com>
Cc: Julien Rouhaud <rjuju123@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: list of extended statistics on psql
Date: Fri, 8 Jan 2021 01:14:39 +0100
Message-ID: <9e76ce7a-e7ab-5f08-9938-59fffc20d106@enterprisedb.com> (raw)
In-Reply-To: <1045d2cb-7ecd-e6bf-cb50-8eeea89cc4b0@nttcom.co.jp_1>
References: <20210107144713.GA8872@alvherre.pgsql>
<61cba5af-5a58-6bda-610c-420077b780c3@enterprisedb.com>
<1045d2cb-7ecd-e6bf-cb50-8eeea89cc4b0@nttcom.co.jp_1>
On 1/8/21 12:52 AM, Tatsuro Yamada wrote:
> Hi,
>
> On 2021/01/08 0:56, Tomas Vondra wrote:
>> On 1/7/21 3:47 PM, Alvaro Herrera wrote:
>>> On 2021-Jan-07, Tomas Vondra wrote:
>>>
>>>> On 1/7/21 1:46 AM, Tatsuro Yamada wrote:
>>>
>>>>> I overlooked the check for MCV in the logic building query
>>>>> because I created the patch as a new feature on PG14.
>>>>> I'm not sure whether we should do back patch or not. However, I'll
>>>>> add the check on the next patch because it is useful if you decide to
>>>>> do the back patch on PG10, 11, 12, and 13.
>>>>
>>>> BTW perhaps a quick look at the other \d commands would show if
>>>> there are
>>>> precedents. I didn't have time for that.
>>>
>>> Yes, we do promise that new psql works with older servers.
>>>
>>
>> Yeah, makes sense. That means we need add the check for 12 / MCV.
>
>
> Ah, I got it.
> I fixed the patch to work with older servers to add the checking
> versions. And I tested \dX command on older servers (PG10 - 13).
> These results look fine.
>
> 0001:
> Added the check code to handle pre-PG12. It has not MCV and
> pg_statistic_ext_data.
> 0002:
> This patch is the same as the previous patch (not changed).
>
> Please find the attached files.
>
OK, thanks. I'll take a look and probably push tomorrow. FWIW I plan to
squash the patches into a single commit.
regards
--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tomas.vondra@enterprisedb.com, tatsuro.yamada.tf@nttcom.co.jp, alvherre@2ndquadrant.com, tomas.vondra@2ndquadrant.com, michael@paquier.xyz, pavel.stehule@gmail.com, rjuju123@gmail.com, pgsql-hackers@lists.postgresql.org
Subject: Re: list of extended statistics on psql
In-Reply-To: <9e76ce7a-e7ab-5f08-9938-59fffc20d106@enterprisedb.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox