From: | Brian Hirt <bhirt(at)me(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-general(at)postgresql(dot)org |
Subject: | Re: pg_upgrade |
Date: | 2010-09-28 23:11:46 |
Message-ID: | C4222AE3-7E70-4008-9882-ECE71615E0D9@me.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
Bruce,
The applied patch has the same behavior on i686 Ubuntu 10.04. It looks like atol() is just a macro for strtol() in stdio.h. I think you want strtoul() instead of strtol()
when i change str2uint() to use strtoul() pg_upgrade completes without a problem (I still haven't tested the upgrade database, but I expect that will be just fine).
I think it's pretty uncommon for the OID to be that big which is why nobody stumbled onto this. This particular installation has pretty much been reloading development databases non stop for the last year. Also, people tend to initdb a lot when testing and doing development which will keep resetting the oid low.
Thanks for getting this one fixed
--brian
On Sep 28, 2010, at 3:49 PM, Bruce Momjian wrote:
> I have applied the attached patch to HEAD and 9.0.X. Odd I had never
> received a bug report about this before. Good thing it didn't silently
> fail, but it is designed to be very picky.
>
From | Date | Subject | |
---|---|---|---|
Next Message | Eric McKeeth | 2010-09-28 23:17:09 | Re: Exclusion constraint issue |
Previous Message | Bartlomiej Korupczynski | 2010-09-28 22:53:55 | How to handle results with column names clash |