Re: [Patch] Adding CORRESPONDING/CORRESPONDING BY to set operation

From: David Fetter <david(at)fetter(dot)org>
To: 毛瑞嘉 <alanmao94(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [Patch] Adding CORRESPONDING/CORRESPONDING BY to set operation
Date: 2019-08-03 15:56:04
Message-ID: 20190803155603.GP31493@fetter.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jul 30, 2019 at 02:43:05PM -0700, 毛瑞嘉 wrote:
> Hi,
>
>
> I wrote a patch for adding CORRESPONDING/CORRESPONDING BY to set operation.
> It is a task in the todo list. This is how the patch works:
>
>
> I modified transformSetOperationStmt() to get an intersection target list
> which is the intersection of the target lists of the left clause and right
> clause for a set operation statement (sostmt). The intersection target list
> is calculated in transformSetOperationTree() and then I modified the target
> lists of the larg and rarg of sostmt to make them equal to the intersection
> target list. Also, I also changed the target list in pstate->p_rtable in
> order to make it consistent with the intersection target list.
>
>
> I attached the scratch version of this patch to the email. I am not sure
> whether the method used in the patch is acceptable or not, but any
> suggestions are appreciated. I will add tests and other related things to
> the patch if the method used in this patch is acceptable.

I tried adding documentation based on what I could infer about the
behavior of this patch. Is that documentation correct?

Best,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Binguo Bao 2019-08-03 16:11:21 Re: [proposal] de-TOAST'ing using a iterator
Previous Message Daniel Migowski 2019-08-03 15:39:33 Patch to clean Query after rewrite-and-analyze - reduces memusage up to 50% - increases TPS by up to 50%