ExecPrepareExprList and per-query context
Amit Langote <langote_amit_f8@lab.ntt.co.jp>
From: Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>
To: Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2017-04-07T10:05:53Z
Lists: pgsql-hackers
As of b8d7f053c5c, ExecPrepareExprList is (must be?) used instead of ExecPrepareExpr when the caller wants to initialize expressions in a list, for example, FormIndexDatum. ExecPrepareExpr doesn't require the caller to have switched to per-query context, because it itself will. Same is not however true for the new ExecPrepareExprList. That means the List node that it creates might be in a context that is not necessarily per-query context, where it previously would be. That breaks third-party users of FormIndexDatum that rely on the list to have been created in per-query context (pg_bulkload was broken by this). Should ExecPrepareExprList also switch to estate->es_query_cxt? Or maybe ExecPrepareExpr could itself detect that passed-in node is a List and create the list of ExprState nodes by itself. I guess the reason to separate list case is because ExecInitExpr() does not take Lists anymore. Thanks, Amit
Commits
-
Ensure that ExecPrepareExprList's result is all in one memory context.
- dbb2a931478a 10.0 landed