On Sun, May 23, 2021 at 12:25:10PM -0400, Tom Lane wrote:
> However, the more I looked at that code the less I liked it.
> I think the way that compression selection is handled for indexes,
> ie consult default_toast_compression on-the-fly, is *far* saner
> than what is currently implemented for tables. So I think we
> should redefine attcompression as "ID of a compression method
> to use, or \0 to select the prevailing default. Ignored if
> attstorage does not permit the use of compression".
+1
It reminds me of reltablespace, which is stored as 0 to mean the database's
default tablespace.
Also, values are currently retoasted during vacuum full if their column's
current compression method doesn't match the value's old compression.
But it doesn't rewrite the column if the it used to use the default
compression, and the default was changed. I think your idea would handle that.
--
Justin
PS. I just ran into the memory leak that Andres reported and Michael fixed.