| From: | "Kevin Grittner" <Kevin(dot)Grittner(at)wicourts(dot)gov> |
|---|---|
| To: | "Noah Misch" <noah(at)leadboat(dot)com> |
| Cc: | "Itagaki Takahiro" <itagaki(dot)takahiro(at)gmail(dot)com>, "Robert Haas" <robertmhaas(at)gmail(dot)com>, <hanada(at)metrosystems(dot)co(dot)jp>,<pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: SQL/MED - file_fdw |
| Date: | 2011-02-11 16:58:18 |
| Message-ID: | 4D55164A020000250003A850@gw.wicourts.gov |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Noah Misch <noah(at)leadboat(dot)com> wrote:
> I'd say, run them with this patch alone. The important thing is
> to not penalize existing COPY users.
Sounds good.
> Incidentally, the "did you want ... ?" was a genuine question. I
> see very little performance risk here, so the tests could be quite
> cursory, even absent entirely.
>From what I've seen, I tend to agree. If there's a committer ready
to go over this, I would say that it might be worth waiting for the
benchmark results against the patch from the day before yesterday to
be run before "pulling the trigger" on it; but proceed on the basis
that it's a near-certainty that it will test out OK.
-Kevin
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2011-02-11 17:03:31 | Re: Range Types: << >> -|- ops vs empty range |
| Previous Message | Dimitri Fontaine | 2011-02-11 16:56:19 | Re: ALTER EXTENSION UPGRADE, v3 |