Alvaro Herrera wrote:
> I think what I should be doing is the same as the returning stuff: keep
> a tupdesc around, and use a single slot, whose descriptor is changed
> just before the projection.
Yes, this works, though it's ugly. Not any uglier than what's already
there, though, so I think it's okay.
The only thing that I remain unhappy about this patch is the whole
adjust_and_expand_partition_tlist() thing. I fear we may be doing
redundant and/or misplaced work. I'll look into it next week.
0001 should be pretty much ready to push -- adjustments to ExecInsert
and ModifyTableState I already mentioned.
0002 is stuff I would like to get rid of completely -- changes to
planner code so that it better supports functionality we need for
0003.
0003 is the main patch. Compared to the previous version, this one
reuses slots by switching them to different tupdescs as needed.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services