From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Andreas Karlsson <andreas(at)proxel(dot)se> |
Cc: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: bug in citext's upgrade script for parallel aggregates |
Date: | 2016-06-29 15:49:49 |
Message-ID: | CA+Tgmoan4qgnj03-xpzPoSKh8AMbmF-sT5Vv7f-MCGt4yThBNA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Jun 25, 2016 at 3:44 AM, Andreas Karlsson <andreas(at)proxel(dot)se> wrote:
> 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.
Since we've already released beta2, I think we need to do a whole new
extension version. We treated beta1 as a sufficiently-significant
event to mandate a version bump, so we should do the same here.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2016-06-29 15:50:17 | Re: dumping database privileges broken in 9.6 |
Previous Message | Jeff Janes | 2016-06-29 15:48:53 | Re: pgbench unable to scale beyond 100 concurrent connections |