Have log_min_duration_statement = 0 always print duration/statement.
authorBruce Momjian <bruce@momjian.us>
Wed, 8 Oct 2003 03:49:38 +0000 (03:49 +0000)
committerBruce Momjian <bruce@momjian.us>
Wed, 8 Oct 2003 03:49:38 +0000 (03:49 +0000)
Change log line to be "duration:  ms  query:"

Indent multi-line queries with a tab in the server logs.

doc/src/sgml/runtime.sgml
src/backend/tcop/postgres.c
src/backend/utils/error/elog.c
src/backend/utils/misc/guc.c
src/backend/utils/misc/postgresql.conf.sample

index 443cd40387fb5c80ae1824eae6b9b4c160764e16..d1b398425dc76e9913c659ae99a13031201cebc6 100644 (file)
@@ -1,5 +1,5 @@
 <!--
-$Header: /cvsroot/pgsql/doc/src/sgml/runtime.sgml,v 1.210 2003/10/03 19:26:49 tgl Exp $
+$Header: /cvsroot/pgsql/doc/src/sgml/runtime.sgml,v 1.211 2003/10/08 03:49:37 momjian Exp $
 -->
 
 <Chapter Id="runtime">
@@ -1689,16 +1689,16 @@ SET ENABLE_SEQSCAN TO OFF;
        <listitem>
         <para>
          Sets a minimum statement execution time (in milliseconds)
-         above which a statement will be logged.  All SQL statements
-         that run longer than the time specified will be logged together
-         with their actual duration.  Setting this to zero (the default)
-    disables time-based logging.  For example, if you set it
-         to <literal>250</literal> then all SQL statements that run longer
-         than 250ms will be logged.  Enabling this
-         option can be useful in tracking down unoptimized queries in
-         your applications.
-         Only superusers can increase this or set it to zero if this option
-         is set to non-zero by the administrator.
+         for statement to be logged.  All SQL statements
+         that run in the time specified or longer will be logged with
+         their duration.  Setting this to zero will print
+         all queries and their durations.  Minus-one (the default)
+         disables this.  For example, if you set it to 
+         <literal>250</literal> then all SQL statements that run 250ms 
+         or longer will be logged.  Enabling this option can be
+         useful in tracking down unoptimized queries in your applications.
+         Only superusers can increase this or set it to minus-one if this 
+         option is set by the administrator.
         </para>
        </listitem>
       </varlistentry>
index 1d17f977cd86eca1790fd1d8e22cb9869fba45e0..80cc52621327ce5e987c9885a8bda9cefea3e043 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.370 2003/10/04 02:47:04 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.371 2003/10/08 03:49:37 momjian Exp $
  *
  * NOTES
  *   this is the "main" module of the postgres backend and
@@ -694,7 +694,7 @@ exec_simple_query(const char *query_string)
     * because gettimeofday() wasn't called. Similarly,
     * log_statement_stats has to be captured once.
     */
-   if (save_log_duration || save_log_min_duration_statement > 0)
+   if (save_log_duration || save_log_min_duration_statement != -1)
        gettimeofday(&start_t, NULL);
 
    if (save_log_statement_stats)
@@ -940,7 +940,7 @@ exec_simple_query(const char *query_string)
     * Combine processing here as we need to calculate the query duration
     * in both instances.
     */
-   if (save_log_duration || save_log_min_duration_statement > 0)
+   if (save_log_duration || save_log_min_duration_statement != -1)
    {
        long        usecs;
        bool        print_statement;
@@ -957,15 +957,17 @@ exec_simple_query(const char *query_string)
         * Output a duration_statement to the log if the query has exceeded
         * the min duration, or if we are to print all durations.
         */
-       print_statement = (save_log_min_duration_statement > 0 &&
-                          usecs >= save_log_min_duration_statement * 1000);
+       print_statement = (save_log_min_duration_statement == 0 ||
+                          (save_log_min_duration_statement > 0 &&
+                           usecs >= save_log_min_duration_statement * 1000));
 
        if (save_log_duration || print_statement)
            ereport(LOG,
-                   (errmsg("duration(secs): %ld.%06ld%s%s",
-                           (long) (stop_t.tv_sec - start_t.tv_sec),
-                           (long) (stop_t.tv_usec - start_t.tv_usec),
-                           print_statement ? " " : "",
+                   (errmsg("duration: %ld.%03ld ms%s%s",
+                           (long) ((stop_t.tv_sec - start_t.tv_sec) * 1000 +
+                           (stop_t.tv_usec - start_t.tv_usec) / 1000),
+                           (long) (stop_t.tv_usec - start_t.tv_usec) % 1000,
+                           print_statement ? "  query: " : "",
                            print_statement ? query_string : "")));
    }
 
@@ -2655,7 +2657,7 @@ PostgresMain(int argc, char *argv[], const char *username)
    if (!IsUnderPostmaster)
    {
        puts("\nPOSTGRES backend interactive interface ");
-       puts("$Revision: 1.370 $ $Date: 2003/10/04 02:47:04 $\n");
+       puts("$Revision: 1.371 $ $Date: 2003/10/08 03:49:37 $\n");
    }
 
    /*
@@ -3196,3 +3198,4 @@ ShowUsage(const char *title)
 
    pfree(str.data);
 }
+
index 8fd5f41046d1693b86f7815946370e5d68b1b659..8c4774fd6f105393988e15ee99b18c3f3b747d9f 100644 (file)
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/utils/error/elog.c,v 1.123 2003/09/25 06:58:05 petere Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/utils/error/elog.c,v 1.124 2003/10/08 03:49:38 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -145,6 +145,7 @@ static const char *useful_strerror(int errnum);
 static const char *error_severity(int elevel);
 static const char *print_timestamp(void);
 static const char *print_pid(void);
+static char *str_prepend_tabs(const char *str);
 
 
 /*
@@ -1135,6 +1136,8 @@ send_message_to_server_log(ErrorData *edata)
    /* Write to stderr, if enabled */
    if (Use_syslog <= 1 || whereToSendOutput == Debug)
    {
+       char *p = str_prepend_tabs(buf.data);
+
        /*
         * Timestamp and PID are only used for stderr output --- we assume
         * the syslog daemon will supply them for us in the other case.
@@ -1142,7 +1145,8 @@ send_message_to_server_log(ErrorData *edata)
        fprintf(stderr, "%s%s%s",
                Log_timestamp ? print_timestamp() : "",
                Log_pid ? print_pid() : "",
-               buf.data);
+               p);
+       pfree(p);
    }
 
    pfree(buf.data);
@@ -1450,3 +1454,24 @@ print_pid(void)
    snprintf(buf, sizeof(buf), "[%d] ", (int) MyProcPid);
    return buf;
 }
+
+/*
+ * str_prepend_tabs
+ *
+ * This string prepends a tab to message continuation lines.
+ */
+static char *str_prepend_tabs(const char *str)
+{
+   char *outstr = palloc(strlen(str) * 2 + 1);
+   int len = strlen(str);
+   int i, outlen = 0;
+
+   for (i = 0; i < len; i++)
+   {
+       outstr[outlen++] = str[i];
+       if (str[i] == '\n' && str[i+1] != '\0' )
+           outstr[outlen++] = '\t';
+   }
+   outstr[outlen++]    = '\0';
+   return outstr;
+}
index 377993a37421558967335141b7f0ca8f82589789..8e2c8517e544834bd1f5e91c8ade154f9c5a999d 100644 (file)
@@ -10,7 +10,7 @@
  * Written by Peter Eisentraut <peter_e@gmx.net>.
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/utils/misc/guc.c,v 1.162 2003/10/03 19:26:49 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/utils/misc/guc.c,v 1.163 2003/10/08 03:49:38 momjian Exp $
  *
  *--------------------------------------------------------------------
  */
@@ -128,7 +128,7 @@ int         log_min_error_statement = PANIC;
 int            log_min_messages = NOTICE;
 int            client_min_messages = NOTICE;
 
-int            log_min_duration_statement = 0;
+int            log_min_duration_statement = -1;
 
 
 /*
@@ -1185,10 +1185,10 @@ static struct config_int ConfigureNamesInt[] =
        {"log_min_duration_statement", PGC_USERLIMIT, LOGGING_WHEN,
            gettext_noop("minimum execution time in milliseconds above which statements will "
                         "be logged"),
-           gettext_noop("The default is 0 (turning this feature off).")
+           gettext_noop("Zero prints all queries.  The default is -1 (turning this feature off).")
        },
        &log_min_duration_statement,
-       0, 0, INT_MAX / 1000, NULL, NULL
+       -1, -1, INT_MAX / 1000, NULL, NULL
    },
 
    /* End-of-list marker */
index c880dc4031818843619f93c12eef6bc22ce38cd9..1ead89e130844f153f47147840e6c06368bbce0f 100644 (file)
                 #   debug5, debug4, debug3, debug2, debug1,
                 #   info, notice, warning, error, panic(off)
                 
-#log_min_duration_statement =  # Log all statements whose
+#log_min_duration_statement = -1 # Log all statements whose
                 # execution time exceeds the value, in
-                # milliseconds.  Zero disables.
+                # milliseconds.  Zero prints all queries.
+                # Minus-one disables.
 
 #silent_mode = false        # DO NOT USE without Syslog!