Re: pgsql: Remove WITH OIDS support, change oid catalog columnvisibility. - Mailing list pgsql-committers

From Peter Eisentraut
Subject Re: pgsql: Remove WITH OIDS support, change oid catalog columnvisibility.
Date
Msg-id ba819888-63c6-7f98-6acb-3731142d9414@2ndquadrant.com
Whole thread Raw
In response to pgsql: Remove WITH OIDS support, change oid catalog column visibility.  (Andres Freund <andres@anarazel.de>)
Responses Re: pgsql: Remove WITH OIDS support, change oid catalog columnvisibility.  (Andres Freund <andres@anarazel.de>)
List pgsql-committers
On 2018-11-21 01:07, Andres Freund wrote:
> Remove WITH OIDS support, change oid catalog column visibility.

I think you may have accidentally duplicated a line in this patch:

@@ -1602,20 +1602,9 @@ ExecFetchSlotHeapTupleDatum(TupleTableSlot *slot)
 void
 ExecInitResultTypeTL(PlanState *planstate)
 {
-   bool        hasoid;
-   TupleDesc   tupDesc;
-
-   if (ExecContextForcesOids(planstate, &hasoid))
-   {
-       /* context forces OID choice; hasoid is now set correctly */
-   }
-   else
-   {
-       /* given free choice, don't leave space for OIDs in result tuples */
-       hasoid = false;
-   }
+   TupleDesc   tupDesc = ExecTypeFromTL(planstate->plan->targetlist);

-   tupDesc = ExecTypeFromTL(planstate->plan->targetlist, hasoid);
+   tupDesc = ExecTypeFromTL(planstate->plan->targetlist);
    planstate->ps_ResultTupleDesc = tupDesc;
 }

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-committers by date:

Previous
From: Michael Meskes
Date:
Subject: pgsql: Add DECLARE STATEMENT support to ECPG.
Next
From: Andrew Gierth
Date:
Subject: pgsql: Fix previous MinGW fix.