On 11/10/2016 09:33 AM, David Steele wrote:
On 11/10/16 10:28 AM, Stephen Frost wrote:

diff --git a/src/backend/access/transam/xlog.c 
b/src/backend/access/transam/xlog.c
[...]
+                       if (log_checkpoints)
+                               ereport(LOG, (errmsg("checkpoint skipped")));

Do we really need to log that we're skipping a checkpoint..?  As the
point of this is to avoid write activity on a system which is idle, it
doesn't make sense to me to add a new cause for writes to happen when
we're idle.

log_checkpoints is not enabled by default, though, so if the user does
enable it don't you think they would want to know when checkpoints
*don't* happen?

Yes but I don't know that it needs to be anywhere below DEBUG2 (vs log_checkpoints).

Sincerely,

JD



--
Command Prompt, Inc.                  http://the.postgres.company/
                        +1-503-667-4564
PostgreSQL Centered full stack support, consulting and development.
Everyone appreciates your honesty, until you are honest with them.
Unless otherwise stated, opinions are my own.


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to