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

Поиск
Список
Период
Сортировка
Искать
От
Ildar Musin
Тема
Re: using index or check in ALTER TABLE SET NOT NULL
Дата
Msg-id
dbfac595-11a0-606f-b67d-ea4a0d542aae@postgrespro.ru
Ответ на
Список
Дерево обсуждения
using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Stephen Frost <sfrost@snowman.net>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Stephen Frost <sfrost@snowman.net>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Stephen Frost <sfrost@snowman.net>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Ildar Musin <i.musin@postgrespro.ru>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Ildar Musin <i.musin@postgrespro.ru>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Alvaro Herrera <alvherre@alvh.no-ip.org>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Ildar Musin <i.musin@postgrespro.ru>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Dmitry Dolgov <9erthalion6@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Dmitry Dolgov <9erthalion6@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Alvaro Herrera <alvherre@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Dmitry Dolgov <9erthalion6@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Dmitry Dolgov <9erthalion6@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Justin Pryzby <pryzby@telsasoft.com>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: Re: using index or check in ALTER TABLE SET NOT NULL David Steele <david@pgmasters.net>
Re: Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Andres Freund <andres@anarazel.de>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL David Rowley <david.rowley@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Tomas Vondra <tomas.vondra@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Tom Lane <tgl@sss.pgh.pa.us>
Re: using index or check in ALTER TABLE SET NOT NULL Ildar Musin <i.musin@postgrespro.ru>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Re: using index or check in ALTER TABLE SET NOT NULL Sergei Kornilov <sk@zsrv.org>
Re: using index or check in ALTER TABLE SET NOT NULL Robert Haas <robertmhaas@gmail.com>
Re: using index or check in ALTER TABLE SET NOT NULL Stephen Frost <sfrost@snowman.net>
Re: using index or check in ALTER TABLE SET NOT NULL Andres Freund <andres@anarazel.de>
Hello Sergei,

On 10.03.2018 12:35, Sergei Kornilov wrote:
> Hello My patch does not apply after commit
> 5748f3a0aa7cf78ac6979010273bd9d50869bb8e. Here is update to current
> master. Not null constraint is immutable too, so here is no changes
> in PartConstraintImpliedByRelConstraint excepts rename and comments
> fix.
>
> In this patch version i also revert tests to v4 state: i use DEBUG
> ereport instead INFO and code path not tested. Please tell me if i
> must change tests some way.
>
> regards, Sergei
>

Ok, I can't think of any other ways to test it so I have to agree with
Tom Lane i.e. rely only on coverage. (There also were another suggestion
to use statistics [select seq_scan from pg_stat_user_tables where
relid='test'::regclass] which show number of table scans. But statistics
is collected by stat collector process with some latency and hence
cannot be reliable for tests).

Patch seems correct to me, it applies and compiles cleanly, docs
compiles as well, tests pass. Changed status to Ready for Committer.

-- 
Ildar Musin
i.musin@postgrespro.ru

В списке pgsql-hackers по дате отправления
От: Tomas Vondra
Дата:
От: Jeremy Finzel
Дата:
FAQ