pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>
Cc: Noah Misch <noah@leadboat.com>
Cc: Adam, Etienne (Nokia-TECH/Issy Les Moulineaux) <etienne.adam@nokia.com>
Cc: PostgreSQL Bugs <pgsql-bugs@postgresql.org>
Cc: Duquesne, Pierre (Nokia-TECH/Issy Les Moulineaux) <pierre.duquesne@nokia.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: [BUGS] [postgresql 10 beta3] unrecognized node type: 90
Date: Mon, 28 Aug 2017 22:17:20 -0400
Message-ID: <9314.1503973040@sss.pgh.pa.us> (raw)
In-Reply-To: <CA+TgmoYnszKkA9ibeRV0fTOT2f9h6ROE5Cf_DZLXEtCpiEF9Tg@mail.gmail.com>
References: <CAL7LBcYM-32B=n_-h_wPHWiML26UvqP3b_WxD7MgXgOwmxvTFw@mail.gmail.com>
	<11673.1502467154@sss.pgh.pa.us>
	<20170815025243.GA3986579@rfd.leadboat.com>
	<CAA4eK1KeQWZOoDmDmGMwuqzPW9JhRS+ditQVFdAfGjNmMZzqMQ@mail.gmail.com>
	<CA+TgmoaWPtfdcZjg+dqLephgKaG7kF8DEdvB870A2t4cUh5x+Q@mail.gmail.com>
	<26053.1502804769@sss.pgh.pa.us>
	<CAA4eK1J_dRv+pGt2bquwmwd_K9+Nght2=tbAc3uUzCinJTMmLg@mail.gmail.com>
	<2941.1502979564@sss.pgh.pa.us>
	<CAA4eK1KeGhogrkO1JWo=WnvEHm_i9tz6-Gt65zELB1hi=P28Hw@mail.gmail.com>
	<CAA4eK1+qD0yfh-iNDKrM7=qMxPOd2KkCZRTX+uxYsDMn0RBwUQ@mail.gmail.com>
	<28782.1502992247@sss.pgh.pa.us>
	<29467.1502993219@sss.pgh.pa.us>
	<CA+Tgmob+S=c=wqaJXNgKT2VhhAVRYFz=MagoTDWb0MueM1xBxg@mail.gmail.com>
	<18650.1502995942@sss.pgh.pa.us>
	<1105.1503008416@sss.pgh.pa.us>
	<24667.1503865781@sss.pgh.pa.us>
	<CAA4eK1LqHdSL0nH04EJ79AnUeKCHPf-rpgkrYszYpKETVFM-Wg@mail.gmail.com>
	<3924.1503923486@sss.pgh.pa.us>
	<CAA4eK1+eEZ5+4jMdWtGtAGK9kDHDcH7Q_xNSxXZV_0wwhTPbZQ@mail.gmail.com>
	<CA+TgmoabrmBs6WaJ! tnrkdyAnmcGUcNbsxm=H43Vs9Xj54BCWUA@mail.gmail.com>
	<16978.1503946802@sss.pgh.pa.us>
	<CA+Tgmob-jDKc8DUTzsCsQEmX7JNqHQC9u3=059WaO95YLwoaAA@mail.gmail.com>
	<300.1503959717@sss.pgh.pa.us>
	<CA+TgmoYnszKkA9ibeRV0fTOT2f9h6ROE5Cf_DZLXEtCpiEF9Tg@mail.gmail.com>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>

Robert Haas <robertmhaas@gmail.com> writes:
> On Mon, Aug 28, 2017 at 6:35 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Maybe parallel_aware should have more than two values, depending
>> on whether the result of the node is context-dependent or not.

> It seems likely the whole concept of parallel_aware is only only a
> zero-order approximation to what we really want.

Yeah, I agree --- but it's also clear that we don't yet know what it
should be.  We'll have to work that out as we accrete more functionality.

In the meantime, I think what we should do is commit the bug fix more or
less as I have it, and then work on Amit's concern about losing parallel
efficiency by separating the resetting of shared parallel-scan state
into a new plan tree traversal that's done before launching new worker
processes.  The only real alternative is to lobotomize the existing rescan
optimizations, and that seems like a really poor choice from here.

			regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers



view thread (44+ messages)  latest in thread

Message-ID: <9314.1503973040@sss.pgh.pa.us>
Permalink:  ../9314.1503973040@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/9314.1503973040@sss.pgh.pa.us

 · 

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-bugs@postgresql.org
  Cc: tgl@sss.pgh.pa.us, robertmhaas@gmail.com, amit.kapila16@gmail.com, noah@leadboat.com, etienne.adam@nokia.com, pierre.duquesne@nokia.com, pgsql-hackers@postgresql.org
  Subject: Re: [BUGS] [postgresql 10 beta3] unrecognized node type: 90
  In-Reply-To: <9314.1503973040@sss.pgh.pa.us>

* 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