| 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: | Whole Thread | Raw Message | 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
| 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 |