From: | Shigeru HANADA <shigeru(dot)hanada(at)gmail(dot)com> |
---|---|
To: | Kohei KaiGai <kaigai(at)kaigai(dot)gr(dot)jp> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Move postgresql_fdw_validator into dblink |
Date: | 2012-10-09 08:56:44 |
Message-ID: | 5073E6CC.3050304@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
(2012/10/09 0:30), Kohei KaiGai wrote:
> The attached patch is a revised one according to my previous
> suggestion. It re-defines "PQconninfoOption *options" as static
> variable with NULL initial value, then, PQconndefaults() shall
> be invoked at once. The default options never changed during
> duration of the backend process, so here is no reason why we
> allocate and free this object for each validator invocation.
Sorry for delayed response. It seems reasonable, so I just fixed
obsolete comment and indent. Please see attached v3 patch which was
rebased against latest head of master.
> If it is also OK for you, I'd like to take over this patch to comitter.
> This patch is prerequisite of postgresql_fdw, so I hope this patch
> getting merged soon.
Please go ahead. :-)
Regards,
--
Shigeru HANADA
Attachment | Content-Type | Size |
---|---|---|
dblink_fdw_validator.v3.patch | text/plain | 32.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Shigeru HANADA | 2012-10-09 09:05:45 | Re: FDW for PostgreSQL |
Previous Message | Sébastien Lardière | 2012-10-09 08:33:23 | Truncate if exists |