From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Remove WITH OIDS support, change oid catalog column visibility. |
Date: | 2019-02-18 19:12:14 |
Message-ID: | 20190218191214.ipkyaetw4mal4pcu@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 2019-02-16 11:14:21 +0100, Peter Eisentraut wrote:
> 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;
> }
Indeed! Thanks for noticing. Fixed.
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2019-02-19 02:24:57 | pgsql: Fix omissions in ecpg/test/sql/.gitignore. |
Previous Message | Andres Freund | 2019-02-18 19:11:53 | pgsql: Remove line duplicated during conflict resolution. |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2019-02-18 19:12:30 | Re: automatically assigning catalog toast oids |
Previous Message | Tom Lane | 2019-02-18 17:43:00 | Re: 2019-03 CF Summary / Review - Tranche #2 |