| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | "K S, Sandhya (Nokia - IN/Bangalore)" <sandhya(dot)k_s(at)nokia(dot)com> |
| Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, "Itnal, Prakash (Nokia - IN/Bangalore)" <prakash(dot)itnal(at)nokia(dot)com> |
| Subject: | Re: Postgres abort found in 9.3.11 |
| Date: | 2016-09-06 11:34:23 |
| Message-ID: | 13741.1473161663@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
"K S, Sandhya (Nokia - IN/Bangalore)" <sandhya(dot)k_s(at)nokia(dot)com> writes:
> I was able to find a patch file where there is a call to ExitPostmaster() in postmaster.c .
> @@ -3081,6 +3081,11 @@
> shmem_exit(1);
> reset_shared(PostPortNumber);
> + /* recovery termination */
> + ereport(FATAL,
> + (errmsg("recovery termination due to process crash")));
> + ExitPostmaster(99);
> +
> StartupPID = StartupDataBase();
> Assert(StartupPID != 0);
> pmState = PM_STARTUP;
There's no such code in the community sources, and I can't say that
such a patch looks like a bright idea to me. It would disable any
restart after a crash (not only during recovery).
If you're running a version with assorted random non-community patches,
we can't really offer much support for that.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David Steele | 2016-09-06 11:52:45 | Re: Proposal for changes to recovery.conf API |
| Previous Message | Rajkumar Raghuwanshi | 2016-09-06 11:15:29 | Re: Declarative partitioning - another take |