Re: duplicate connection failure messages
Bruce Momjian <bruce@momjian.us>
From: Bruce Momjian <bruce@momjian.us>
To: Peter Eisentraut <peter_e@gmx.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Alvaro Herrera <alvherre@commandprompt.com>, Magnus Hagander <magnus@hagander.net>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2010-11-26T16:53:23Z
Lists: pgsql-hackers
Attachments
- /rtmp/diff (text/x-diff) patch
Peter Eisentraut wrote: > On l?r, 2010-11-20 at 18:07 -0500, Bruce Momjian wrote: > > The output is as expected: > > > > $ psql -h localhost test > > psql: could not connect to server: Connection refused > > Is the server running on host "localhost" (127.0.0.1) and accepting > > TCP/IP connections on port 5432? > > $ psql -h 127.0.0.1 test > > psql: could not connect to server: Connection refused > > Is the server running on host "127.0.0.1" and accepting > > TCP/IP connections on port 5432? > > Thanks for working on this. However, the example I posted at the > beginning of this thread now does this: > > $ ./psql -p 55555 -h localhost > psql: could not connect to server: Connection refused > Is the server running on host "localhost" (???) and accepting > TCP/IP connections on port 55555? > could not connect to server: Connection refused > Is the server running on host "localhost" (127.0.0.1) and accepting > TCP/IP connections on port 55555? > > The "???" should presumably be "::1". OK, I updated the code to always use cur_addr in the code --- let me know if that doesn't fix it. > Also, this comment should be updated: > > /* > * Try to initiate a connection to one of the addresses > * returned by pg_getaddrinfo_all(). conn->addr_cur is the > * next one to try. We fail when we run out of addresses > * (reporting the error returned for the *last* alternative, > * which may not be what users expect :-(). > */ Thanks, comment udpated. It was wrong even before because we were reporting all failures even before I Started. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. +