]> granicus.if.org Git - postgresql/commit
Avoid using INFO elevel for what are fundamentally debug messages.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 7 Sep 2019 23:03:11 +0000 (19:03 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 7 Sep 2019 23:03:11 +0000 (19:03 -0400)
commit121e3ceed6d93c2891208005f0b270ee80dc13e7
treeb6d8c49794c507f4ac9df9ace52213dc8ccf48d2
parentb6a4d0498757a6e345e07b2a84eb8333f5e5235e
Avoid using INFO elevel for what are fundamentally debug messages.

Commit 6f6b99d13 stuck an INFO message into the fast path for
checking partition constraints, for no very good reason except
that it made it easy for the regression tests to verify that
that path was taken.  Assorted later patches did likewise,
increasing the unsuppressable-chatter level from ALTER TABLE
even more.  This isn't good for the user experience, so let's
drop these messages down to DEBUG1 where they belong.  So as
not to have a loss of test coverage, create a TAP test that
runs the relevant queries with client_min_messages = DEBUG1
and greps for the expected messages.

This testing method is a bit brute-force --- in particular,
it duplicates the execution of a fair amount of the core
create_table and alter_table tests.  We experimented with
other solutions, but running any significant amount of
standard testing with client_min_messages = DEBUG1 seems
to have a lot of output-stability pitfalls, cf commits
bbb96c370 and 5655565c0.  Possibly at some point we'll look
into whether we can reduce the amount of test duplication.

Backpatch into v12, because some of these messages are new
in v12 and we don't really want to ship it that way.

Sergei Kornilov

Discussion: https://postgr.es/m/81911511895540@web58j.yandex.ru
Discussion: https://postgr.es/m/4859321552643736@myt5-02b80404fd9e.qloud-c.yandex.net
src/backend/commands/tablecmds.c
src/backend/partitioning/partbounds.c
src/test/modules/Makefile
src/test/modules/test_misc/.gitignore [new file with mode: 0644]
src/test/modules/test_misc/Makefile [new file with mode: 0644]
src/test/modules/test_misc/README [new file with mode: 0644]
src/test/modules/test_misc/t/001_constraint_validation.pl [new file with mode: 0644]
src/test/regress/expected/alter_table.out
src/test/regress/expected/create_table.out
src/tools/msvc/Mkvcbuild.pm