| From: | Mithun Cy <mithun(dot)cy(at)enterprisedb(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Kuntal Ghosh <kuntalghosh(dot)2007(at)gmail(dot)com>, Catalin Iacob <iacobcatalin(at)gmail(dot)com>, "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>, Craig Ringer <craig(at)2ndquadrant(dot)com>, Peter van Hardenberg <pvh(at)pvh(dot)ca>, Peter Eisentraut <peter_e(at)gmx(dot)net>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Patch: Implement failover on libpq connect level. |
| Date: | 2016-12-05 04:00:21 |
| Message-ID: | CAD__Ouh1WZ3K+cYf7q4mdE9ASmEgBYR7y20+F6X6haWJCB9c9A@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-jdbc |
On Fri, Dec 2, 2016 at 8:54 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> Couldn't this just be a variable in PQconnectPoll(), instead of adding
> a new structure member?
I have fixed same with a local variable in PQconnectPoll, Initally I
thought savedMessage should have same visiblitly as errorMessage so we can
restore the errorMessage even outside PQconnectPoll. But that seems not
required. Attacting the new patch which fixes the same.
--
Thanks and Regards
Mithun C Y
EnterpriseDB: http://www.enterprisedb.com
| Attachment | Content-Type | Size |
|---|---|---|
| failover-to-new-master-bug-fix-02.patch | application/octet-stream | 4.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Haribabu Kommi | 2016-12-05 05:03:31 | Re: [PATCH] Generic type subscription |
| Previous Message | Michael Paquier | 2016-12-05 03:21:59 | Re: Proposal for changes to recovery.conf API |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2016-12-05 17:53:13 | Re: Patch: Implement failover on libpq connect level. |
| Previous Message | Robert Haas | 2016-12-02 15:24:30 | Re: Patch: Implement failover on libpq connect level. |