On Tue, 2008-04-01 at 15:34 +0200, Albe Laurenz wrote:
> This follows up on the discussion in
> - pg_ctl will refuse a smart shutdown during online backup.
> - The postmaster will also refuse to shutdown in smart mode
> in that case and log a message to that effect.
> - In fast shutdown mode, the server will rename "backup_label"
> after successfully shutting down and log the fact.
* smart shutdown waits for sessions to complete, yet this just ignores
smart shutdowns which is something a little different. I think we
should wait for the backup to complete and then shutdown.
* when we say "online backup cancelled" I think we should say something
more like "online backup mode cancelled". All we are doing is removing
the backup label file, we're not actually cancelling the physical backup
since it is external to the database anyway.
* The #defines at top of postmaster.c are duplicated from xlog.c
If we can't agree on a common header file then we should at least add a
comment to mention they are duplicated (in both locations).
PostgreSQL UK 2008 Conference: http://www.postgresql.org.uk
In response to
pgsql-hackers by date
|Next:||From: Aidan Van Dyk||Date: 2008-04-01 16:53:56|
|Subject: Re: Several tags around PostgreSQL 7.1 broken|
|Previous:||From: David Fetter||Date: 2008-04-01 16:35:03|
|Subject: Re: Cast as compound type|
pgsql-patches by date
|Next:||From: Simon Riggs||Date: 2008-04-01 19:03:18|
|Subject: Re: Improve shutdown during online backup|
|Previous:||From: Simon Riggs||Date: 2008-04-01 16:17:34|
|Subject: Re: [HACKERS] Minor changes to Recovery related code|