On Fri, Feb 24, 2017 at 1:24 PM, Robert Haas <robertmh...@gmail.com> wrote: > On Fri, Feb 24, 2017 at 7:29 PM, Kohei KaiGai <kai...@kaigai.gr.jp> wrote: >> This attached patch re-designed the previous v2 according to Robert's >> comment. >> All I had to do is putting entrypoint for ForeignScan/CustomScan at >> ExecShutdownNode, >> because it is already modified to call child node first, earlier than >> ExecShutdownGather >> which releases DSM segment. > > Aside from the documentation, which needs some work, this looks fine > to me on a quick read.
LGTM too. You may want to clarify in the docs when the hook is called in relation to other hooks too. -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers