From: "Joe Conway" <m...@joeconway.com>
Fair enough -- this patch does it at that level in
materializeQueryResult()

I'm in favor of this. TBH, I did this at first, but I was afraid this would be rejected due to the reason mentioned earlier.

if statement in PG_TRY block is not necessary like this, because sinfo is zero-cleared.


  PG_TRY();
  {
+   /* Create short-lived memory context for data conversions */
+   sinfo.tmpcontext = AllocSetContextCreate(CurrentMemoryContext,
+            "dblink temporary context",

Regards
MauMau



--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to