Re: BUG in pg_autovacuum - with patch - Mailing list pgsql-bugs

From Karl Denninger
Subject Re: BUG in pg_autovacuum - with patch
Date
Msg-id 20050402010754.A47501@denninger.net
Whole thread Raw
In response to Re: BUG in pg_autovacuum - with patch  ("Matthew T. O'Connor" <matthew@zeut.net>)
Responses Re: BUG in pg_autovacuum - with patch
List pgsql-bugs
I have a process which does backups by mounting a disk into a RAID array,
allowing it to sync, then it must stop the database before detaching it so
as to insure that the DBMS is consistent on the backup disk.

Once detached, Postgres must be restarted, of course......

This process is MUCH faster than dumping the disks to tape and results in a
bootable backup volume - the latter is of great value for disaster recovery!

--
--
Karl Denninger (karl@denninger.net) Internet Consultant & Kids Rights Activist
http://www.denninger.net    My home on the net - links to everything I do!
http://scubaforum.org        Your UNCENSORED place to talk about DIVING!
http://www.spamcuda.net        SPAM FREE mailboxes - FREE FOR A LIMITED TIME!
http://genesis3.blogspot.com    Musings Of A Sentient Mind

On Sat, Apr 02, 2005 at 01:50:03AM -0500, Matthew T. O'Connor wrote:
> Why would you run pg_autovacuum from cron?
>
> Karl Denninger wrote:
>
> >Hi folks.
> >
> >There's a problem in the console detach code for pg_autovacuum.
> >
> >"setsid()" is NOT sufficient.  Specifically, you must disassociate the
> >control terminal (standard in, out and error) for it to be complete.
> >
> >This causes any attempt to execute pg_autovacuum from the cron to hang the
> >cron processor, stalling it.
> >
> >Find below a diff that fixes the problem and allows proper disassociation:
> >
> >
> >*** pg_autovacuum.c.old    Fri Apr  1 21:13:22 2005
> >--- pg_autovacuum.c    Fri Apr  1 20:55:41 2005
> >***************
> >*** 196,201 ****
> >--- 196,202 ----
> >  daemonize(void)
> >  {
> >      pid_t        pid;
> >+     int        i;
> >
> >      pid = fork();
> >      if (pid == (pid_t) -1)
> >***************
> >*** 219,224 ****
> >--- 220,230 ----
> >          fflush(LOGOUTPUT);
> >          _exit(1);
> >      }
> >+     i = open(NULL_DEV, O_RDWR);
> >+         dup2(i, 0);
> >+         dup2(i, 1);
> >+         dup2(i, 2);
> >+         close(i);
> >  #endif
> >
> >  }
> >
> >
> >--
> >
> >
>
>
>
> %SPAMBLOCK-SYS: Matched [@postgresql.org], message ok

pgsql-bugs by date:

Previous
From: "Matthew T. O'Connor"
Date:
Subject: Re: BUG in pg_autovacuum - with patch
Next
From: "Matthew T. O'Connor"
Date:
Subject: Re: BUG in pg_autovacuum - with patch