Add comment about rationale for continuing to use C library functions

instead of src/timezone for timestamping log entries.

diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c
index dc881a0067b9f9e8f71ba44b0ff5534ddd3a4943..e47740ededad028f2fcfc44dd898f5bf69d2ed18 100644 (file)
--- a/src/backend/utils/error/elog.c
+++ b/src/backend/utils/error/elog.c
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/error/elog.c,v 1.140 2004/06/03 02:08:04 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/error/elog.c,v 1.141 2004/06/21 14:12:38 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1214,6 +1214,16 @@ log_line_prefix(StringInfo buf)
                break;
            case 't':
                {
+                   /*
+                    * Note: for %t and %s we deliberately use the C library's
+                    * strftime/localtime, and not the equivalent functions
+                    * from src/timezone.  This ensures that all backends
+                    * will report log entries in the same timezone, namely
+                    * whatever C-library setting they inherit from the
+                    * postmaster.  If we used src/timezone then local
+                    * settings of the TimeZone GUC variable would confuse
+                    * the log.
+                    */
                    time_t stamp_time = time(NULL);
                    char strfbuf[128];