Re: Archiver not picking up changes to archive_command

From: bricklen <bricklen(at)gmail(dot)com>
To: Greg Smith <greg(at)2ndquadrant(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-general(at)postgresql(dot)org
Subject: Re: Archiver not picking up changes to archive_command
Date: 2010-05-11 01:45:38
Message-ID: AANLkTilD1DXbigG2bRVALhcEIvCg63X3n2nN5CQ-4Y68@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers

On Mon, May 10, 2010 at 6:12 PM, Greg Smith <greg(at)2ndquadrant(dot)com> wrote:
> Tom Lane wrote:
>>
>> A look at the code shows that the archiver only notices SIGHUP once
>> per outer loop, so the change would only take effect once you catch up,
>> which is not going to help much in this case.  Possibly we should change
>> it to check for SIGHUP after each archive_command execution.
>>
>
> I never considered this a really important issue to sort out because I tell
> everybody it's unwise to put something complicated directly into
> archive_command.  Much better to call a script that gets passed %f/%p, then
> let that script do all the work; don't even have to touch the server config
> if you need to fix something then.  The lack of error checking that you get
> when just writing some shell commands directly in the archive_command itself
> horrifies me in a production environment.
>
> --
> Greg Smith  2ndQuadrant US  Baltimore, MD
> PostgreSQL Training, Services and Support
> greg(at)2ndQuadrant(dot)com   www.2ndQuadrant.us

Thanks Greg, that's a good idea. I'll revise that series of commands
into a script, and add some error handling as you suggest.

Cheers,

Bricklen

In response to

Browse pgsql-general by date

  From Date Subject
Next Message Mike Christensen 2010-05-11 02:00:40 Re: peer-to-peer replication with Postgres
Previous Message bricklen 2010-05-11 01:44:09 Re: Archiver not picking up changes to archive_command

Browse pgsql-hackers by date

  From Date Subject
Next Message Carlos Jordão 2010-05-11 03:19:48 Invitation to connect on LinkedIn
Previous Message bricklen 2010-05-11 01:44:09 Re: Archiver not picking up changes to archive_command