pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Fabien COELHO <coelho@cri.ensmp.fr>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Thomas Munro <thomas.munro@gmail.com>
Cc: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: Allow an alias to be attached directly to a JOIN ... USING
Date: Tue, 24 Dec 2019 19:13:32 +0100 (CET)
Message-ID: <alpine.DEB.2.21.1912241908490.27864@pseudo> (raw)
In-Reply-To: <20190917173742.GA8735@alvherre.pgsql>
References: <20190917173742.GA8735@alvherre.pgsql>


On Tue, 17 Sep 2019, Alvaro Herrera wrote:

>> Indeed, that seems like a problem, and it's a good question.  You can
>> see this on unpatched master with SELECT x.filler FROM
>> (pgbench_tellers AS t JOIN b USING (bid)) AS x.
>
> I'm not sure I understand why that problem is a blocker for this patch.

As discussed on another thread,

     https://www.postgresql.org/message-id/flat/2aa57950-b1d7-e9b6-0770-fa592d565dda@2ndquadrant.com

the patch does not conform to spec

   SQL:2016 Part 2 Foundation Section 7.10 <joined table>

Basically "x" is expected to include *ONLY* joined attributes with USING, 
i.e. above only x.bid should exists, and per-table aliases are expected to 
still work for other attributes.

ISTM that this patch could be "returned with feedback".

-- 
Fabien.





view thread (21+ messages)  latest in thread

Message-ID: <alpine.DEB.2.21.1912241908490.27864@pseudo>
Permalink:  ../alpine.DEB.2.21.1912241908490.27864@pseudo/
Also on:    postgresql.org/message-id/alpine.DEB.2.21.1912241908490.27864@pseudo

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: coelho@cri.ensmp.fr, alvherre@2ndquadrant.com, thomas.munro@gmail.com, peter.eisentraut@2ndquadrant.com
  Subject: Re: Allow an alias to be attached directly to a JOIN ... USING
  In-Reply-To: <alpine.DEB.2.21.1912241908490.27864@pseudo>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox