On 2019-Jan-11, Amit Langote wrote:
> Looking around a bit more, I started thinking even build_child_join_sjinfo
> doesn't belong in appendinfo.c (just to be clear, it was defined in
> prepunion.c before this commit), so maybe we should move it to joinrels.c
> and instead export adjust_child_relids that's required by it from
> appendinfo.c. There's already adjust_child_relids_multilevel in
> appendinfo.h, so having adjust_child_relids next to it isn't too bad. At
> least not as bad as appendinfo.c exporting build_child_join_sjinfo for
> joinrels.c to use.
OK, pushed, thanks. I may have caused merge conflicts with the rest of
the series, because I reordered some functions -- sorry about that.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services