On 1/7/21 1:46 AM, Tatsuro Yamada wrote:
> Hi Tomas,
>
>> Thanks, and Happy new year to you too.
>>
>> I almost pushed this, but I have one more question. listExtendedStats
>> first checks the server version, and errors out for pre-10 servers.
>> Shouldn't the logic building query check the server version too, to
>> decide whether to check the MCV stats? Otherwise it won't work on 10
>> and 11, which does not support the "mcv" stats.
>>> I don't recall what exactly is our policy regarding new psql with
>>> older server versions, but it seems strange to check for 10.0 and
>>> then fail anyway for "supported" versions.
>
> Thanks for your comments.
>
> 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.
>
+1
BTW perhaps a quick look at the other \d commands would show if there
are precedents. I didn't have time for that.
> I wonder the column names added by \dX+ is fine? For example,
> "Ndistinct_size" and "Dependencies_size". It looks like long names,
> but acceptable?
>
Seems acceptable - I don't have a better idea.
regards
--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company