Re: bug in citext's upgrade script for parallel aggregates

From: Andreas Karlsson <andreas(at)proxel(dot)se>
To: David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: bug in citext's upgrade script for parallel aggregates
Date: 2016-06-25 07:44:18
Message-ID: a3677a09-ed76-e3b1-23e2-19a1c02173a7@proxel.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 06/24/2016 01:31 PM, David Rowley wrote:
> Seems there's a small error in the upgrade script for citext for 1.1
> to 1.2 which will cause min(citext) not to be parallel enabled.
>
> max(citext)'s combinefunc is first set incorrectly, but then updated
> to the correct value. I assume it was meant to set the combine
> function for min(citext) instead.
>
> Fix attached. I've assumed that because we're still in beta that we
> can get away with this fix rather than making a 1.3 version to fix the
> issue.

Yes, this is indeed a bug.

Andreas

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Julien Rouhaud 2016-06-25 08:57:18 Re: Rename max_parallel_degree?
Previous Message Amit Kapila 2016-06-25 07:33:44 Re: Rename max_parallel_degree?