Re: memory context for tuplesort return values

Tom Lane wrote:
> Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> > Crazy ideas: add a #warning or #error to the header file unless there is
> > some special symbol previously defined, something like
> > #define I_UNDERSTAND_MEM_ALLOC_IN_TUPLETABLESLOT
> > which means the developer did update his code.
>
> Well, if we wanted to go that far, the simple solution is to rename
> tuplesort_gettuple to something else. Guaranteed code breakage then.
> However, I think this is overkill for a problem that only *might*
> bite people. I kinda doubt that people are using TupleTableSlots in
> add-on code, because the design of the executor makes it hard for
> anything except plan nodes to obtain slots.

Yeah, you may be right. I guess I was worried because the Mammoth
Replicator code uses TupleTableSlots here and there and I was assuming
that other developers may have similar code. However, Replicator is
fairly involved stuff, and I don't see that TupleTableSlots would have
much use outside something that tied to the internals. And of course,
every major Postgres release is major surgery for Replicator as well, so
nothing new here :-(

--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.