Re: Push down more full joins in postgres_fdw

From: Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>
To: Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Push down more full joins in postgres_fdw
Date: 2016-11-01 12:01:57
Message-ID: cc63e3b7-eb72-f21f-7067-08d0ecb823bc@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox
Thread:
Lists: pgsql-hackers

On 2016/10/27 20:41, Etsuro Fujita wrote:
> Here is the updated version, which includes the restructuring you
> proposed. Other than the above issue and the alias issue we discussed,
> I addressed all your comments except one on testing; I tried to add test
> cases where the remote query is deparsed as nested subqueries, but I
> couldn't because IIUC, reduce_outer_joins reduced full joins to inner
> joins or left joins. So, I added two test cases: (1) the joining
> relations are both base relations (actually, we already have that) and
> (2) one of the joining relations is a base relation and the other is a
> join relation. I rebased the patch to HEAD, so I added a test case with
> aggregate pushdown also.

I've updated another patch for evaluating PHVs remotely. Attached is an
updated version of the patch, which is created on top of the patch for
supporting deparsing subqueries. You can find test cases where a remote
join query is deparsed as nested subqueries.

Best regards,
Etsuro Fujita

Attachment Content-Type Size
postgres-fdw-phv-pushdown-v3.patch application/x-patch 72.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2016-11-01 12:19:41 Re: commit fest manager for CF 2016-11?
Previous Message Rafia Sabih 2016-11-01 11:44:41 Re: pgbench - allow backslash continuations in \set expressions