From: Tom Lane <tgl@sss.pgh.pa.us>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
cc: James Coleman <jtc331@gmail.com>,
        Tomas Vondra <tomas.vondra@2ndquadrant.com>,
        Michael Paquier <michael@paquier.xyz>,
        Rafia Sabih <rafia.pghackers@gmail.com>, Peter Geoghegan <pg@bowt.ie>,
        Simon Riggs <simon@2ndquadrant.com>,
        Shaun Thomas <shaun.thomas@2ndquadrant.com>,
        Dmitry Dolgov <9erthalion6@gmail.com>,
        Alexander Korotkov <a.korotkov@postgrespro.ru>,
        PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>,
        andreas@proxel.se
Subject: Re: [PATCH] Incremental sort (was: PoC: Partial sort)
In-reply-to: <20200323164446.GA6911@alvherre.pgsql>
References: <20200323164446.GA6911@alvherre.pgsql>
Comments: In-reply-to Alvaro Herrera <alvherre@2ndquadrant.com>
	message dated "Mon, 23 Mar 2020 13:44:46 -0300"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <18326.1584983119.1@sss.pgh.pa.us>
Date: Mon, 23 Mar 2020 13:05:19 -0400
Message-ID: <18327.1584983119@sss.pgh.pa.us>
Precedence: bulk

Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> ... all plan types that use only one child use the outer one.  They
> could use either, as long as it does that consistently, I think.

Yeah, exactly.  The outer/inner terminology is really only sensible
for join nodes, but there isn't a third child-plan pointer reserved
for single-child node types, so you gotta use one of those.  And
conventionally we use the "outer" one.

			regards, tom lane