* Robert Haas (robertmh...@gmail.com) wrote:
> On Wed, Jan 12, 2011 at 12:25 PM, Stephen Frost <sfr...@snowman.net> wrote:
> > Attached is ...
> 
> I don't see an attachment, other than signature.asc.

I suck, sorry about that, here it is..

See, inlining is better!  I wouldn't have forgotten it! ;)

        Stephen
*** a/src/backend/utils/error/elog.c
--- b/src/backend/utils/error/elog.c
***************
*** 3,8 ****
--- 3,17 ----
   * elog.c
   *	  error logging and reporting
   *
+  * A few comments about situations where error processing is called:
+  *
+  * We need to be cautious of both a performance hit when logging, since
+  * log messages can be generated at a huge rate if every command is being
+  * logged and we also need to watch out for what can happen when we are
+  * trying to log from an aborted transaction.  Specifically, attempting to
+  * do SysCache lookups and possibly use other usually available backend
+  * systems will fail badly when logging from an aborted transaction.
+  *
   * Some notes about recursion and errors during error processing:
   *
   * We need to be robust about recursive-error scenarios --- for example,

Attachment: signature.asc
Description: Digital signature

Reply via email to