Remove bogus set_ps_display call --- changing displayed status here is

either wrong or unnecessary in most cases, and on systems where setting
status takes a kernel call, the overhead of setting status three times
per command rather than two is annoying.

diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c
index 8ffa33f539ec683248014541dfca6eecd720093a..fbee4470c98e833e825dcba47d9038b7702b2a2d 100644 (file)
--- a/src/backend/tcop/postgres.c
+++ b/src/backend/tcop/postgres.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.206 2001/01/24 19:43:09 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.207 2001/02/18 04:28:31 tgl Exp $
  *
  * NOTES
  *   this is the "main" module of the postgres backend and
@@ -878,7 +878,7 @@ finish_xact_command(void)
    /* Now commit the command */
    if (DebugLvl >= 1)
        elog(DEBUG, "CommitTransactionCommand");
-   set_ps_display("commit");   /* XXX probably the wrong place to do this */
+
    CommitTransactionCommand();
 
 #ifdef SHOW_MEMORY_STATS
@@ -1680,7 +1680,7 @@ PostgresMain(int argc, char *argv[], int real_argc, char *real_argv[], const cha
    if (!IsUnderPostmaster)
    {
        puts("\nPOSTGRES backend interactive interface ");
-       puts("$Revision: 1.206 $ $Date: 2001/01/24 19:43:09 $\n");
+       puts("$Revision: 1.207 $ $Date: 2001/02/18 04:28:31 $\n");
    }
 
    /*
@@ -1781,6 +1781,11 @@ PostgresMain(int argc, char *argv[], int real_argc, char *real_argv[], const cha
         */
        ReadyForQuery(whereToSendOutput);
 
+       if (IsTransactionBlock())
+           set_ps_display("idle in transaction");
+       else
+           set_ps_display("idle");
+
        /* ----------------
         *   (2) deal with pending asynchronous NOTIFY from other backends,
         *   and enable async.c's signal handler to execute NOTIFY directly.
@@ -1791,10 +1796,6 @@ PostgresMain(int argc, char *argv[], int real_argc, char *real_argv[], const cha
 
        EnableNotifyInterrupt();
 
-       if (!IsTransactionBlock())
-           set_ps_display("idle");
-       else    set_ps_display("idle in transaction");
-
        /* Allow "die" interrupt to be processed while waiting */
        ImmediateInterruptOK = true;
        /* and don't forget to detect one that already arrived */