On 2017/09/07 6:52, Tom Lane wrote:
Tatsuro Yamada writes:
The declaration of postgresGetForeignPlan uses baserel, but
the actual definition uses foreignrel. It would be better to sync.
Pushed, thanks.
regards, tom lane
Thanks!
Regards,
Tatsuro Yamada
--
Sent v
Tatsuro Yamada writes:
> The declaration of postgresGetForeignPlan uses baserel, but
> the actual definition uses foreignrel. It would be better to sync.
Pushed, thanks.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make cha
Hi Fujita-san,
> The patch looks good to me, so I'll mark this as Ready for Committer.
Thanks for reviewing my patch.
> (I'm not sure we should do the same thing to the function declaration in
> other places such as fdwapi.h and the documentation for consistency, but if
> so, I'd vote for lea
On 2017/04/07 13:12, Tatsuro Yamada wrote:> The declaration of
postgresGetForeignPlan uses baserel, but
the actual definition uses foreignrel. It would be better to sync.
Agreed.
Please find attached a patch.
The patch looks good to me, so I'll mark this as Ready for Committer.
(I'm not s
Hi,
The declaration of postgresGetForeignPlan uses baserel, but
the actual definition uses foreignrel. It would be better to sync.
Please find attached a patch.
Tatsuro Yamada
NTT Open Source Software Center
diff --git a/contrib/postgres_fdw/postgres_fdw.c b/contrib/postgres_fdw/postgres_fdw.c
i
