Re: [ADMIN] pg_upgrade from 9.1.3 to 9.2 failed - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [ADMIN] pg_upgrade from 9.1.3 to 9.2 failed
Date
Msg-id 20120924130820.GB21242@momjian.us
Whole thread Raw
In response to Re: [ADMIN] pg_upgrade from 9.1.3 to 9.2 failed  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: [ADMIN] pg_upgrade from 9.1.3 to 9.2 failed  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Mon, Sep 24, 2012 at 09:06:04AM -0400, Peter Eisentraut wrote:
> On 9/24/12 8:55 AM, Bruce Momjian wrote:
> > I can confirm that pg_upgrade does case-insensitive comparisons of
> > encoding/locale names:
> > 
> >     static void
> >     check_locale_and_encoding(ControlData *oldctrl,
> >                               ControlData *newctrl)
> >     {
> >         /* These are often defined with inconsistent case, so use pg_strcasecmp(). */
> >         if (pg_strcasecmp(oldctrl->lc_collate, newctrl->lc_collate) != 0)
> >             pg_log(PG_FATAL,
> >                    "old and new cluster lc_collate values do not match\n");
> >         if (pg_strcasecmp(oldctrl->lc_ctype, newctrl->lc_ctype) != 0)
> >             pg_log(PG_FATAL,
> >                    "old and new cluster lc_ctype values do not match\n");
> 
> I seem to recall that at some point in the distant past, somehow some
> Linux distributions changed the canonical spelling of locale names from
> xx_YY.UTF-8 to xx_YY.utf8.  So if people are upgrading old PostgreSQL
> instances that use the old spelling, pg_upgrade will probably fail.  A
> fix might be to take the locale name you find in pg_control and run it
> through setlocale() to get the new canonical name.

Or we could just remove dashes from the name before comparisons.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: [ADMIN] pg_upgrade from 9.1.3 to 9.2 failed
Next
From: Amit Kapila
Date:
Subject: Re: Switching timeline over streaming replication