Re: Postgres_fdw join pushdown - wrong results with whole-row reference

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>
Cc: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Postgres_fdw join pushdown - wrong results with whole-row reference
Date: 2016-07-01 14:15:00
Message-ID: CA+TgmobZczziUDsvis7q2_g8GX3+j5neaYfKjLe+3j2q056U3w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jun 28, 2016 at 8:20 AM, Ashutosh Bapat
<ashutosh(dot)bapat(at)enterprisedb(dot)com> wrote:
>> > postgres_fdw resets the search path to pg_catalog while opening
>> > connection
>> > to the server. The reason behind this is explained in deparse.c
>> >
>> > * We assume that the remote session's search_path is exactly
>> > "pg_catalog",
>> > * and thus we need schema-qualify all and only names outside
>> > pg_catalog.
>>
>> Hmm. OK, should we revert the schema-qualification part of that
>> commit, or just leave it alone?
>
> If we leave that code as is, someone who wants to add similar code later
> would get confused or will be tempted to create more instances of
> schema-qualification. I think we should revert the schema qualification.

OK, done.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2016-07-01 14:16:23 Re: Postgres_fdw join pushdown - wrong results with whole-row reference
Previous Message Tom Lane 2016-07-01 14:11:09 Re: EXISTS clauses not being optimized in the face of 'one time pass' optimizable expressions