Re: Move pg_attribute.attcompression to earlier in struct for reduced size? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Move pg_attribute.attcompression to earlier in struct for reduced size?
Date
Msg-id 1677426.1622075390@sss.pgh.pa.us
Whole thread Raw
In response to Re: Move pg_attribute.attcompression to earlier in struct for reduced size?  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Michael Paquier <michael@paquier.xyz> writes:
> Yeah, having an extra test for partitioned tables would be a good
> idea.

We do have some coverage already via the pg_upgrade test.

> Could it be possible to have some tests for COMPRESSION DEFAULT?  It
> seems to me that this should be documented as a supported keyword for
> CREATE/ALTER TABLE.

Uh, I did do both of those, no?  (The docs treat "default" as another
possible value, not a keyword, even though it's a keyword internally.)

>  --changing column storage should not impact the compression method
>  --but the data should not be compressed
>  ALTER TABLE cmdata2 ALTER COLUMN f1 TYPE varchar;
> +ALTER TABLE cmdata2 ALTER COLUMN f1 SET COMPRESSION pglz;
> This comment needs a refresh?

It's correct AFAICS.  Maybe it needs a bit of editing for clarity,
but I'm not sure how to make it better.  The point is that the
SET STORAGE just below disables compression of new values, no
matter what SET COMPRESSION says.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Move pg_attribute.attcompression to earlier in struct for reduced size?
Next
From: Andres Freund
Date:
Subject: Re: Move pg_attribute.attcompression to earlier in struct for reduced size?