]> granicus.if.org Git - postgresql/commit
Shut down Gather's children before shutting down Gather itself.
authorRobert Haas <rhaas@postgresql.org>
Wed, 22 Feb 2017 02:29:27 +0000 (07:59 +0530)
committerRobert Haas <rhaas@postgresql.org>
Wed, 22 Feb 2017 02:38:07 +0000 (08:08 +0530)
commitacf555bc53acb589b5a2827e65d655fa8c9adee0
tree78ba8654993bf1c9dc443b2b7e29c5ac421d1a1d
parentd912dd062b64287adcabab4180abafefd07cea14
Shut down Gather's children before shutting down Gather itself.

It turns out that the original shutdown order here does not work well.
Multiple people attempting to develop further parallel query patches
have discovered that they need to do cleanup before the DSM goes away,
and you can't do that if the parent node gets cleaned up first.

Patch by me, reviewed by KaiGai Kohei and Dilip Kumar.

Discussion: http://postgr.es/m/CA+TgmoY6bOc1YnhcAQnMfCBDbsJzROQ3sYxSAL-SYB5tMJcTKg@mail.gmail.com
Discussion: http://postgr.es/m/9A28C8860F777E439AA12E8AEA7694F8012AEB82@BPXM15GP.gisp.nec.co.jp
Discussion: http://postgr.es/m/CA+TgmoYuPOc=+xrG1v0fCsoLbKAab9F1ddOeaaiLMzKOiBar1Q@mail.gmail.com
src/backend/executor/execProcnode.c
src/backend/executor/nodeGather.c