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
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 |
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 |