From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Small improvement to parallel query docs |
Date: | 2017-02-13 20:31:01 |
Message-ID: | CA+TgmoZ0G5oCN8tVGycyf56VOOTWdOodKXTs1aRnaMCZGsV0Og@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Feb 13, 2017 at 3:29 PM, David Rowley
<david(dot)rowley(at)2ndquadrant(dot)com> wrote:
> On 14 February 2017 at 09:21, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> On Sun, Feb 12, 2017 at 7:16 PM, David Rowley
>> - table. Each worker will execute the outer side of the plan in full, which
>> - is why merge joins are not supported here. The outer side of a merge join
>> - will often involve sorting the entire inner table; even if it involves an
>> - index, it is unlikely to be productive to have multiple processes each
>> - conduct a full index scan of the inner table.
>> + relation. Each worker will execute the inner side of the join in full,
>> + which is why merge joins are not supported here. The inner side of a merge
>> + join will often involve sorting the entire inner relation; even if it
>> + involves an index, it is unlikely to be productive to have multiple
>> + processes each conduct a full index scan of the inner side of the join.
>>
>> Why s/table/relation/? I don't think that's useful, especially
>> because the first part of that very same paragraph would still say
>> "table".
>
> Perhaps it's not correct to do that. I did mean relation is the true
> relational theory sense, rather than where relkind = 'r'. I didn't
> really like the way it assumed the inner side was a table. Perhaps its
> better to just say "involve sorting the entire inner side of the join"
Yeah, that seems fine.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2017-02-13 20:40:38 | Re: \if, \elseif, \else, \endif (was Re: PSQL commands: \quit_if, \quit_unless) |
Previous Message | David Rowley | 2017-02-13 20:29:33 | Re: Small improvement to parallel query docs |