psql \conninfo command (was: Patch: psql \whoami option)

From: David Christensen <david(at)endpoint(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Steve Singer <ssinger_pg(at)sympatico(dot)ca>, pgsql-hackers Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: psql \conninfo command (was: Patch: psql \whoami option)
Date: 2010-07-18 18:00:03
Message-ID: D59F974B-365E-4BCD-92D1-5B763F4482DC@endpoint.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On Jul 18, 2010, at 12:33 PM, David Christensen wrote:

>
> On Jul 18, 2010, at 12:30 PM, Tom Lane wrote:
>
>> David Christensen <david(at)endpoint(dot)com> writes:
>>> machack:machack:5432=# \c "foo""bar"
>>> You are now connected to database "foo"bar".
>>
>> What this is reflecting is that backslash commands have their own weird
>> rules for processing double quotes. What I was concerned about was that
>> double quotes in SQL are normally used for protecting mixed case, and
>> you don't need that for \c:
>>
>> regression=# create database "FooBar";
>> CREATE DATABASE
>> regression=# \c foobar
>> FATAL: database "foobar" does not exist
>> Previous connection kept
>> regression=# \c FooBar
>> You are now connected to database "FooBar".
>> FooBar=#
>>
>> The fact that there are double quotes around the database name in the
>> "You are now connected..." message is *not* meant to imply that that is
>> a valid double-quoted SQL identifier, either. It's just an artifact of
>> how we set off names in English-language message style. In another
>> language it might look like <<FooBar>> or some such.
>>
>> My opinion remains that you should just print the user and database
>> names as-is, without trying to inject any quoting into the mix. You're
>> more likely to confuse people than help them if you do that.
>
>
> Okay, understood. Then consider my updated patch (just sent attached to a recent message) to reflect the desired behavior. (I'll update the commitfest patch entry when it shows up in the archives.)

Updated the commitfest entry with the patch, updated the title to reflect the actual name of the command, and marked as ready for committer.

Regards,

David
--
David Christensen
End Point Corporation
david(at)endpoint(dot)com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dimitri Fontaine 2010-07-18 18:39:07 Re: SHOW TABLES
Previous Message David Christensen 2010-07-18 17:33:51 Re: Patch: psql \whoami option