| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | Chris Ernst <cernst(at)zvelo(dot)com> |
| Cc: | pgsql-general(at)postgresql(dot)org |
| Subject: | Re: pg_upgrade not detecting version properly |
| Date: | 2012-10-10 17:54:49 |
| Message-ID: | 20121010175449.GG11892@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-general pgsql-hackers |
On Wed, Oct 10, 2012 at 10:35:06AM -0600, Chris Ernst wrote:
> On 10/10/2012 09:56 AM, Bruce Momjian wrote:
> > Can you show me what is in the PG_VERSION file in the old cluster? It
> > should be "9.1".
>
> Hi Bruce,
>
> Thank you for the reply. Indeed it is "9.1":
>
> # cat /postgresql/9.1/main/PG_VERSION
> 9.1
>
> And just for good measure:
>
> cat /postgresql/9.2/main/PG_VERSION
> 9.2
>
> And there are no other PostgreSQL versions on this machine.
>
> Hmm... I was just about to send this when something else occurred to me.
> I had initially tried to run pg_upgrade as root and it said it couldn't
> be run as root. So I've been running it as my own user (which is in the
> postgres group). However, everything in /postgresql/9.1/main is owned
> by postgres with 700 permissions.
>
> I switched to the postgres user and now pg_upgrade is running. Perhaps
> just a more informative error message is in order.
>
> Thank you for the shove in the right direction =)
Oops, that code was returning zero if it couldn't open the file. The
attached, applied patch to head and 9.2 issues a proper error message.
Seems this "zero return" has been in the code since the beginning. :-(
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
| Attachment | Content-Type | Size |
|---|---|---|
| open.diff | text/x-diff | 929 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alban Hertroys | 2012-10-10 18:11:38 | Re: pymssql Connection to the database failed for an unknown reason |
| Previous Message | Greg Sabino Mullane | 2012-10-10 17:45:33 | Re: Planner chooses multi-column index in 9.2 when maybe it should not |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2012-10-10 17:57:31 | Re: Minor document updates |
| Previous Message | Daniel Frey | 2012-10-10 17:32:16 | Bug / feature request for floating point to string conversion |