Re: using index or check in ALTER TABLE SET NOT NULL

Sergei Kornilov <sk@zsrv.org>

From: Sergei Kornilov <sk@zsrv.org>

To: Stephen Frost <sfrost@snowman.net>

Cc: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>

Date: 2018-01-23T19:06:07Z

Lists: pgsql-hackers

Attachments

alter_table_set_not_null_by_constraints_only_v4.patch (text/x-diff) patch v4

Hello Stephen

I changed the suggested things and added some regression tests. Also i wrote few words to the documentation. New patch is attached.

Thank you for feedback!
regards, Sergei

Commits

Avoid using INFO elevel for what are fundamentally debug messages.
- db438318997b 13.0 landed
- 121e3ceed6d9 12.0 landed
Revert setting client_min_messages to 'debug1' in new tests.
- 5655565c077c 12.0 landed
Allow ALTER TABLE .. SET NOT NULL to skip provably unnecessary scans.
- bbb96c3704c0 12.0 landed
Improve predtest.c's internal docs, and enhance its functionality a bit.
- 5748f3a0aa7c 11.0 cited