Skip site navigation (1) Skip section navigation (2)

Re: psql 8.4 \c repeats version banner

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: psql 8.4 \c repeats version banner
Date: 2010-02-06 17:35:42
Message-ID: 201002061735.o16HZgw12992@momjian.us (view raw or flat)
Thread:
Lists: pgsql-hackers
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(at)momjian(dot)us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

Attachment: /pgpatches/psql_banner
Description: text/x-diff (2.9 KB)

In response to

Responses

pgsql-hackers by date

Next:From: Tom LaneDate: 2010-02-06 18:04:09
Subject: Re: pg_class has no toast table?
Previous:From: Andres FreundDate: 2010-02-06 16:39:07
Subject: Re: Hot Standby and DROP DATABASE

Privacy Policy | About PostgreSQL
Copyright © 1996-2013 The PostgreSQL Global Development Group