Prev: [COMMITTERS] pgsql: pgindent run for 9.0, second run
Next: psql \conninfo command (was: Patch: psql \whoami option)
From: Robert Haas on 21 Jul 2010 22:57 On Wed, Jul 21, 2010 at 9:48 PM, Fujii Masao <masao.fujii(a)gmail.com> wrote: > On Wed, Jul 21, 2010 at 7:29 PM, Robert Haas <robertmhaas(a)gmail.com> wrote: >> On Wed, Jul 21, 2010 at 1:07 AM, Fujii Masao <masao.fujii(a)gmail.com> wrote: >>> On Tue, Jul 20, 2010 at 11:14 PM, Robert Haas <robertmhaas(a)gmail.com> wrote: >>>> OK, committed. >>> >>> When I specify the path of the directory for the Unix-domain socket >>> as the host, \conninfo doesn't mention that this connection is based >>> on the Unix-domain socket. Is this intentional? >>> >>> $ psql -h"/tmp" -c"\conninfo" >>> You are connected to database "postgres" on host "/tmp" at port "5432" >>> as user "postgres". >>> >>> I expected that something like >>> >>> � �You are connected to database "postgres" via local socket on >>> "/tmp" at port "5432" as user "postgres". >> >> :-( >> >> No, I didn't realize the host field could be used that way. �It's true >> that you get a fairly similar message from \c, but that's not exactly >> intuitive either. >> >> rhaas=# \c - - /tmp - >> You are now connected to database "rhaas" on host "/tmp". > > OK. The attached patch makes \conninfo command emit the following > message if the host begins with a slash: > > � �$ psql -h/tmp -c"\conninfo" > � �You are connected to database "postgres" via local socket on > "/tmp" at port "5432" as user "postgres". > > Similarly, it makes \c command emit the following message in that > case: > > � �$ psql -hlocalhost -c"\c - - /tmp -" > � �You are now connected to database "postgres" via local socket on "/tmp". > > Comments? Should we be using is_absolute_path() here instead, as libpq does? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company -- Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers |