Avoid calling pq_flush just after sending AUTH_REQ_OK; this saves one

send() and probable process context swap during backend startup.

diff --git a/src/backend/libpq/auth.c b/src/backend/libpq/auth.c
index df9091e9b461c8e5cfd0316ec7a64764ba6846c1..e3dd02cb4890f66f59ae11db0df7c870936cf2dd 100644 (file)
--- a/src/backend/libpq/auth.c
+++ b/src/backend/libpq/auth.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/libpq/auth.c,v 1.72 2001/11/05 17:46:25 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/libpq/auth.c,v 1.73 2002/02/19 19:49:09 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -619,9 +619,16 @@ sendAuthRequest(Port *port, AuthRequest areq)
                pq_sendbytes(&buf, port->cryptSalt, 2);
 
        pq_endmessage(&buf);
-       pq_flush();
+
+       /*
+        * Flush message so client will see it, except for AUTH_REQ_OK,
+        * which need not be sent until we are ready for queries.
+        */
+       if (areq != AUTH_REQ_OK)
+               pq_flush();
 }
 
+
 #ifdef USE_PAM
 
 /*