Re: psql 8.4 \c repeats version banner - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: psql 8.4 \c repeats version banner
Date
Msg-id 201002162107.o1GL73d02304@momjian.us
Whole thread Raw
In response to Re: psql 8.4 \c repeats version banner  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
Applied.

---------------------------------------------------------------------------

Bruce Momjian wrote:
> Peter Eisentraut wrote:
> > In 8.3, running \c from a file prints something like
> > 
> > You are now connected to database "postgres".
> > 
> > In 8.4 it prints
> > 
> > psql (8.4.1)
> > You are now connected to database "postgres".
> > 
> > Is it intentional/sensible to repeat the startup banner every time the
> > connection changes, or was this unintentionally introduced while the
> > startup banner was reshuffled in 8.4?
> 
> I did some reseach on this.  I bet this behavior was added when we
> decided to print the backend version warning banner on \c as well as
> startup, because it is possible for the backend to be different version
> from the backend originally used for psql startup.  The code that prints
> the psql banner and the warning banner are in the same function and
> share the same output line.
> 
> What I did in the attached patch is to add a boolean to
> connection_warnings() to indicate whether it was being called on psql
> startup or via \c, and to supress the psql banner on \c if the client
> and server versions match:
> 
>     $ psql test
>     psql (8.5devel)
>     Type "help" for help.
>     
>     test=> \c test
>     You are now connected to database "test".
>     test=>
> 
> Any version mismatch will still print the psql banner for \c, which is
> what I think we want.
> 
> -- 
>   Bruce Momjian  <bruce@momjian.us>        http://momjian.us
>   EnterpriseDB                             http://enterprisedb.com
> 
>   + If your life is a hard drive, Christ can be your backup. +


> 
> -- 
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +


pgsql-hackers by date:

Previous
From: Robert Doerfler
Date:
Subject: Re: OpenVMS?
Next
From: Tom Lane
Date:
Subject: Re: LISTEN/NOTIFY and notification timing guarantees