Re: Small issues with CREATE TABLE COMPRESSION
Dilip Kumar <dilipbalaut@gmail.com>
From: Dilip Kumar <dilipbalaut@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Michael Paquier <michael@paquier.xyz>, Postgres hackers <pgsql-hackers@lists.postgresql.org>,
Jacob Champion <pchampion@vmware.com>
Date: 2021-05-05T08:11:03Z
Lists: pgsql-hackers
On Wed, May 5, 2021 at 11:02 AM Dilip Kumar <dilipbalaut@gmail.com> wrote: > > On Wed, May 5, 2021 at 12:06 AM Robert Haas <robertmhaas@gmail.com> wrote: > > > > > > There are no tests in pg_dump to make sure that some ALTER > > > MATERIALIZED VIEW or ALTER TABLE commands are generated when the > > > compression of a matview's or table's column is changed. > > > > True, but it does seem to work. I am happy if you or anyone want to > > write some tests. > > I think it will be really hard to generate such a test in pg_dump, > because default we are compiling --without-lz4, which means we have > only one compression option available, and if there is only one option > available then the column compression method and the default > compression method will be same so the dump will not generate an extra > command of type ALTER TABLE... SET COMPRESSION. I think we already have such test cases at least through pg_upgrade. Basically, if you see in compression.sql we are not dropping the table so that pg_upgrade and dump them and test. So if test run --with-lz4 then in pg_upgrade dump we can see ALTER TABLE... SET COMPRESSION type of commands. -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com
Commits
-
Add support for LZ4 build in MSVC scripts
- 9ca40dcd4d0c 14.0 landed
-
Add more TAP tests for pg_dump with attribute compression
- 63db0ac3f9e6 14.0 landed
-
doc: Fix some gaps with the documentation related to LZ4
- 02a93e7ef961 14.0 landed
-
Fix incorrect error code for CREATE/ALTER TABLE COMPRESSION
- 9681f2160dcb 14.0 landed
-
Additional doc fixes for configurable TOAST compression.
- 448b02c00515 14.0 landed
-
docs: Clarify how ALTER TABLE .. SET COMPRESSION works.
- 2d0f66240263 14.0 landed