Re: Add support for logging the current role

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Add support for logging the current role
Date: 2011-02-14 02:52:51
Message-ID: 20110214025251.GG4116@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andrew,

* Andrew Dunstan (andrew(at)dunslane(dot)net) wrote:
> See the discussion back around the the 8.1 release (IIRC) when we
> added the HEADER option.

I recall seeing it and not agreeing with it then. :)

> >If I wanted something to throw away the first record of a file before
> >loading it, I'd use tail.
>
> The whole point of us having direct CSV import is to minimise the
> requirement for preprocessing.

Having a 'throw away 1 line' option is just silly to me, but documenting
it as "header" is worse. Water under the bridge at this point though.

> That said, I think there's probably a good case for an option to use
> the header line as a column list. I know of at least one application
> I have written that could benefit from it. But that's work for 9.2
> or later.

I agree it's work for 9.2. I could probably help with this if people
agree that it should be done.

Thanks,

Stephen

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message YAMAMOTO Takashi 2011-02-14 03:08:24 Re: SSI bug?
Previous Message Stephen Frost 2011-02-14 02:51:08 Re: Add support for logging the current role