Don't append epoch to log_filename if no format specifier is given.

Robert Haas

diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 223911c7c682004a7c3a8d54c25d32962b49cf0a..2ceeaf7a19d2c10847d3f40f3b20944295bf615f 100644 (file)
--- a/doc/src/sgml/config.sgml
+++ b/doc/src/sgml/config.sgml
@@ -2398,16 +2398,20 @@ local0.*    /var/log/postgresql
         so <literal>%</literal>-escapes can be used to specify time-varying
         file names.  (Note that if there are
         any time-zone-dependent <literal>%</literal>-escapes, the computation
-        is done in the zone specified by <xref linkend="guc-log-timezone">.)
-        If no <literal>%</literal>-escapes are present,
-        <productname>PostgreSQL</productname> will append the epoch of the new
-        log file's creation time.  For example, if
-        <varname>log_filename</varname> were <literal>server_log</literal>,
-        then the chosen file name would be <literal>server_log.1093827753</>
-        for a log starting at Sun Aug 29 19:02:33 2004 MST.
+        is done in the zone specified
+        by <xref linkend="guc-log-timezone">.)
         Note that the system's <systemitem>strftime</systemitem> is not used
         directly, so platform-specific (nonstandard) extensions do not work.
        </para>
+       <para>
+        If you specify a file name without escapes, you should plan to
+        use a log rotation utility to avoid eventually filling the
+        entire disk.  In releases prior to 8.4, if
+        no <literal>%</literal> escapes were
+        present, <productname>PostgreSQL</productname> would append
+        the epoch of the new log file's creation time, but this is no
+        longer the case.
+       </para>
        <para>
         If CSV-format output is enabled in <varname>log_destination</>,
         <literal>.csv</> will be appended to the timestamped 

diff --git a/src/backend/postmaster/syslogger.c b/src/backend/postmaster/syslogger.c
index 988b0e04ff21412ccb0e4cc23c05dce50e04a73e..0f8d93a968b948e514c0fcf890702ba354a705e2 100644 (file)
--- a/src/backend/postmaster/syslogger.c
+++ b/src/backend/postmaster/syslogger.c
@@ -1191,18 +1191,9 @@ logfile_getname(pg_time_t timestamp, char *suffix)
 
        len = strlen(filename);
 
-       if (strchr(Log_filename, '%'))
-       {
-               /* treat it as a strftime pattern */
-               pg_strftime(filename + len, MAXPGPATH - len, Log_filename,
-                                       pg_localtime(&timestamp, log_timezone));
-       }
-       else
-       {
-               /* no strftime escapes, so append timestamp to new filename */
-               snprintf(filename + len, MAXPGPATH - len, "%s.%lu",
-                                Log_filename, (unsigned long) timestamp);
-       }
+       /* treat it as a strftime pattern */
+       pg_strftime(filename + len, MAXPGPATH - len, Log_filename,
+               pg_localtime(&timestamp, log_timezone));
 
        if (suffix != NULL)
        {