pgsql: postgres_fdw: Push down joins to remote servers.

Started by Robert Haasalmost 10 years ago4 messages
#1Robert Haas
rhaas@postgresql.org

postgres_fdw: Push down joins to remote servers.

If we've got a relatively straightforward join between two tables,
this pushes that join down to the remote server instead of fetching
the rows for each table and performing the join locally. Some cases
are not handled yet, such as SEMI and ANTI joins. Also, we don't
yet attempt to create presorted join paths or parameterized join
paths even though these options do get tried for a base relation
scan. Nevertheless, this seems likely to be a very significant win
in many practical cases.

Shigeru Hanada and Ashutosh Bapat, reviewed by Robert Haas, with
additional review at various points by Tom Lane, Etsuro Fujita,
KaiGai Kohei, and Jeevan Chalke.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/e4106b2528727c4b48639c0e12bf2f70a766b910

Modified Files
--------------
contrib/postgres_fdw/deparse.c | 531 +++++++++---
contrib/postgres_fdw/expected/postgres_fdw.out | 1094 ++++++++++++++++++++++--
contrib/postgres_fdw/postgres_fdw.c | 835 ++++++++++++++++--
contrib/postgres_fdw/postgres_fdw.h | 46 +-
contrib/postgres_fdw/sql/postgres_fdw.sql | 213 ++++-
doc/src/sgml/postgres-fdw.sgml | 10 +
6 files changed, 2464 insertions(+), 265 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Robert Haas (#1)
Re: pgsql: postgres_fdw: Push down joins to remote servers.

Robert Haas <rhaas@postgresql.org> writes:

postgres_fdw: Push down joins to remote servers.

The buildfarm is not very impressed with this patch.

regards, tom lane

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#3Robert Haas
robertmhaas@gmail.com
In reply to: Tom Lane (#2)
Re: [COMMITTERS] pgsql: postgres_fdw: Push down joins to remote servers.

On Tue, Feb 9, 2016 at 3:10 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Robert Haas <rhaas@postgresql.org> writes:

postgres_fdw: Push down joins to remote servers.

The buildfarm is not very impressed with this patch.

Well, I guess that's why we have regression tests. It looks like that
test case introduces a dependency on particular CTID values that
wasn't there before. That doesn't seem like a very good idea; I'll
remove that test.

--
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

#4Ashutosh Bapat
ashutosh.bapat@enterprisedb.com
In reply to: Robert Haas (#3)
Re: [COMMITTERS] pgsql: postgres_fdw: Push down joins to remote servers.

Sorry for the trouble. Thanks Robert for fixing it.

On Wed, Feb 10, 2016 at 2:12 AM, Robert Haas <robertmhaas@gmail.com> wrote:

On Tue, Feb 9, 2016 at 3:10 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Robert Haas <rhaas@postgresql.org> writes:

postgres_fdw: Push down joins to remote servers.

The buildfarm is not very impressed with this patch.

Well, I guess that's why we have regression tests. It looks like that
test case introduces a dependency on particular CTID values that
wasn't there before. That doesn't seem like a very good idea; I'll
remove that test.

--
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

--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company