Re: Change ereport level for QueuePartitionConstraintValidation

On 2019-Jul-15, David Rowley wrote:

> I think the only argument against it was around lack of ability to
> test if the constraint was used to verify no row breaks the partition
> bound during the ATTACH PARTITION.

Would it work to set client_min_messages to DEBUG1 for the duration of
the test, or does that have too much unrelated noise?

> Does anyone feel strongly that we need to the test to confirm that the
> constraint was used for this?

Well, IME if we don't test it, we're sure to break it in the future.
The only questions are 1) when, 2) how long till we notice, 3) how
difficult is it to fix at that point.  I think breakage is easily
noticed by users, and a fix is unlikely to require hard measures such as
ABI breaks or catversion bumps.  I'd like more than zero tests, but it
doesn't seem *that* severe.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services