| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Jeff Janes <jeff(dot)janes(at)gmail(dot)com> |
| Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Euler Taveira <euler(at)timbira(dot)com(dot)br>, Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: move collation import to backend |
| Date: | 2017-01-18 16:12:13 |
| Message-ID: | 16636.1484755933@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Jeff Janes <jeff(dot)janes(at)gmail(dot)com> writes:
> With this commit, I'm getting 'make check' fail at initdb with the error:
> 2017-01-18 07:47:50.565 PST [43691] FATAL: collation "aa_ER(at)saaho" for
> encoding "UTF8" already exists
Yeah, so are large chunks of the buildfarm. Having now read the patch,
I see that the problem is that it simply ignored the de-duplication
logic that existed in initdb's implementation. That was put there
on the basis of bitter experience, as I recall.
The new code seems to think it's sufficient to do an "if not exists"
insertion when generating abbreviated names, but that's wrong, and
even if it avoided outright failures, it would be nondeterministic
(I doubt "locale -a" is guaranteed to emit locale names in any
particular order).
I think this needs to be reverted pending redesign of the de-duplication
coding.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Corey Huinker | 2017-01-18 16:24:10 | Re: PSQL commands: \quit_if, \quit_unless |
| Previous Message | Karl O. Pinc | 2017-01-18 16:11:20 | Re: Patch to implement pg_current_logfile() function |