On Mon, Mar 14, 2016 at 4:31 AM, Etsuro Fujita <fujita.ets...@lab.ntt.co.jp> wrote: >> It was left there intentionally to document all the conditions in one >> place (some from the core and some from the FDW itself), for a ready >> reference. In case tomorrow core thinks that matching user mapping is >> not required, postgres_fdw would still require it to be incorporated. > > Thank you for the explanation! I understand the reason, but that seems > confusing to me.
Agreed. >> In addition, I'd like to update some related comments in >> src/include/nodes/relation.h and >> src/backend/optimizer/path/joinpath.c. >> >> >> Those look fine. Sorry for missing those in the commit and thanks for >> providing a patch for the same. > > No problem. Committed. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
