Horton

Thread

Commits

  1. Add macro for maximum statistics target

  2. Change type of pg_statistic_ext.stxstattarget

  3. Take pg_attribute out of VacAttrStats

  1. several attstattarget-related improvements

    Peter Eisentraut <peter@eisentraut.org> — 2023-06-28T14:52:27Z

    Here are a few patches related to attstattarget:

- 0001: Change type of pg_statistic_ext.stxstattarget, to match 
attstattarget.  Maybe this should go into PG16, for consistency?

- 0002: Add macro for maximum statistics target, instead of hardcoding 
it everywhere.

- 0003: Take pg_attribute out of VacAttrStats.  This simplifies some 
code, especially for extended statistics, which had to have weird 
workarounds.

  2. Re: several attstattarget-related improvements

    Tomas Vondra <tomas.vondra@enterprisedb.com> — 2023-06-28T21:30:11Z

    On 6/28/23 16:52, Peter Eisentraut wrote:
> Here are a few patches related to attstattarget:
> 
> - 0001: Change type of pg_statistic_ext.stxstattarget, to match
> attstattarget.  Maybe this should go into PG16, for consistency?
> 
> - 0002: Add macro for maximum statistics target, instead of hardcoding
> it everywhere.
> 
> - 0003: Take pg_attribute out of VacAttrStats.  This simplifies some
> code, especially for extended statistics, which had to have weird
> workarounds.

+1 to all three patches

Not sure about 0001 vs PG16, it'd require catversion bump.


regards

-- 
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

  3. Re: several attstattarget-related improvements

    Tom Lane <tgl@sss.pgh.pa.us> — 2023-06-28T22:10:09Z

    Tomas Vondra <tomas.vondra@enterprisedb.com> writes:
> On 6/28/23 16:52, Peter Eisentraut wrote:
>> - 0001: Change type of pg_statistic_ext.stxstattarget, to match
>> attstattarget.  Maybe this should go into PG16, for consistency?

> Not sure about 0001 vs PG16, it'd require catversion bump.

Yeah, past beta1 I think we should be conservative about bumping
catversion.  Suggest you hold this for now, and if we find some
more-compelling reason for a catversion bump in v16, we can sneak
it in at that time.  Otherwise, I won't cry if it waits for v17.

			regards, tom lane

  4. Re: several attstattarget-related improvements

    Peter Eisentraut <peter@eisentraut.org> — 2023-07-03T05:26:12Z

    On 28.06.23 23:30, Tomas Vondra wrote:
> On 6/28/23 16:52, Peter Eisentraut wrote:
>> Here are a few patches related to attstattarget:
>>
>> - 0001: Change type of pg_statistic_ext.stxstattarget, to match
>> attstattarget.  Maybe this should go into PG16, for consistency?
>>
>> - 0002: Add macro for maximum statistics target, instead of hardcoding
>> it everywhere.
>>
>> - 0003: Take pg_attribute out of VacAttrStats.  This simplifies some
>> code, especially for extended statistics, which had to have weird
>> workarounds.
> 
> +1 to all three patches
> 
> Not sure about 0001 vs PG16, it'd require catversion bump.

committed (to PG17 for now)