forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitc596fad
committed
Generate a separate upper relation for each stage of setop planning.
Commit3fc6e2d made setop planningstages return paths rather than plans, but all such paths were looselyassociated with a single RelOptInfo, and only the final path was addedto the RelOptInfo. Even at the time, it was foreseen that this shouldbe changed, because there is otherwise no good way for a single stageof setop planning to return multiple paths. With this patch, eachstage of set operation planning now creates a separate RelOptInfo;these are distinguished by using appropriate relid sets. Note thatthis patch does nothing whatsoever about actually returning multiplepaths for the same set operation; it just makes it possible for afuture patch to do so.Along the way, adjust things so that create_upper_paths_hook is calledfor each of these new RelOptInfos rather than just once, since thatmight be useful to extensions using that hook. It might be a goodto provide an FDW API here as well, but I didn't try to do that fornow.Patch by me, reviewed and tested by Ashutosh Bapat and RajkumarRaghuwanshi.Discussion:http://postgr.es/m/CA+TgmoaLRAOqHmMZx=ESM3VDEPceg+-XXZsRXQ8GtFJO_zbMSw@mail.gmail.com1 parent49525c4 commitc596fad
1 file changed
+190
-150
lines changed0 commit comments
Comments
(0)