Re: Unified logging system for command-line programs
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2018-12-30T19:45:23Z
Lists: pgsql-hackers
Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes: > I have developed a patch that unifies the various ad hoc logging > (message printing, error printing) systems used throughout the > command-line programs. I've not read the patch in any detail, but +1 for making this more uniform. > - Common files (common/, fe_utils/, etc.) can handle logging much more > simply by just using one API without worrying too much about the context > of the calling program, requiring callbacks, or having to pass > "progname" around everywhere. It seems like a shame that src/common files still need to have #ifdef FRONTEND variant code to deal with frontend vs. backend conventions. I wonder how hard it would be to layer some subset of ereport() functionality on top of what you have here, so as to get rid of those #ifdef stanzas. regards, tom lane
Commits
-
Unified logging system for command-line programs
- cc8d41511721 12.0 landed
-
Assorted translatable string fixes
- 55a70a023c3d 10.0 cited