Re: Change ereport level for QueuePartitionConstraintValidation - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Change ereport level for QueuePartitionConstraintValidation
Date
Msg-id 20190723163559.GA23211@alvherre.pgsql
Whole thread Raw
In response to Re: Change ereport level for QueuePartitionConstraintValidation  (David Rowley <david.rowley@2ndquadrant.com>)
List pgsql-hackers
On 2019-Jul-23, David Rowley wrote:

> Also, if I'm not wrong, the votes so far appear to be:
> 
> NOTICE:  Robert, Amit
> DEBUG1: Tom, Alvaro (I'm entirely basing this on the fact that they
> mentioned possible ways to test with DEBUG1)
> 
> I'll be happy with DEBUG1 if we can get tests to test it.

Well, I think the user doesn't *care* to see a message about the
optimization.  They just want the command to be fast.  *We* (developers)
want the message in order to ensure the command remains fast.  So some
DEBUG level seems the right thing.

Another way to reach the same conclusion is to think about the "building
index ... serially" messages, which are are pretty much in the same
category and are using DEBUG1.  (I do think the TOAST ones are just
noise though, and since they disrupt potential testing with
client_min_messages=debug1, another way to go about this is to reduce
those to DEBUG2 or just elide them.)

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



pgsql-hackers by date:

Previous
From: Justin Pryzby
Date:
Subject: stress test for parallel workers
Next
From: Tom Lane
Date:
Subject: Re: [bug fix] Produce a crash dump before main() on Windows