| From: | Robert Haas <robertmhaas(at)gmail(dot)com> | 
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> | 
| Cc: | Andreas Seltenreich <seltenreich(at)gmx(dot)de>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> | 
| Subject: | Re: [sqlsmith] Failed assertions on parallel worker shutdown | 
| Date: | 2016-06-06 19:18:33 | 
| Message-ID: | CA+TgmoZkJ-cBwy7tjMnKt-CtAew7Tn_yMxy5gP7mii8OUL5W-g@mail.gmail.com | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
On Fri, Jun 3, 2016 at 11:13 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> I think the workers should stop processing tuples after the tuple queues got
>> detached.  This will not only handle above situation gracefully, but will
>> allow to speed up the queries where Limit clause is present on top of Gather
>> node.  Patch for the same is attached with mail (this was part of original
>> parallel seq scan patch, but was not applied and the reason as far as I
>> remember was we thought such an optimization might not be required for
>> initial version).
>
> This is very likely a good idea, but...
>
>> Another approach to fix this issue could be to reset mqh_partial_bytes and
>> mqh_length_word_complete in shm_mq_sendv  in case of SHM_MQ_DETACHED.  These
>> are currently reset only incase of success.
>
> ...I think we should do this too
Patches for both of these things are now committed.  Barring problems
with those commits, I believe this resolves this open item.
-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2016-06-06 19:26:41 | Re: Changed SRF in targetlist handling | 
| Previous Message | Robert Haas | 2016-06-06 19:16:10 | Re: Reviewing freeze map code |