pgsql: Fix dumb bug in tqueue.c

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix dumb bug in tqueue.c
Date: 2015-11-18 13:28:52
Message-ID: E1Zz2n6-0003UB-Ct@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix dumb bug in tqueue.c

When I wrote this code originally, the intention was to recompute the
remapinfo only when the tupledesc changes. This presumably only
happens once per query, but I copied the design pattern from other
DestReceivers. However, due to a silly oversight on my part,
tqueue->tupledesc never got set, leading to recomputation for every
tuple.

This should improve the performance of parallel scans that return a
significant number of tuples.

Report by Amit Kapila; patch by me, reviewed by him.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/adeee974866085db84b860c1f397dd7c6b136a0a

Modified Files
--------------
src/backend/executor/tqueue.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Thom Brown 2015-11-18 13:54:47 Re: [COMMITTERS] pgsql: Add pg_audit, an auditing extension
Previous Message Amit Kapila 2015-11-18 11:51:06 Re: pgsql: Generate parallel sequential scan plans in simple cases.