On Fri, 2004-07-09 at 12:53, Klaus Naumann wrote:
> archive_program is provided with a string which contains the target directory.
> That doesn't really make sense. 

archive_dest is used for both archive and restore, thats why its set as
a separate parameter.

Thats the rationale...lets see what others think

> First of all it introduces the problem you
> mentioned in the README file (if the directory doesn't exist you loose
> xlogs). 

Your example quoted later is the answer....
        archive_dest = '/mnt/pgarch/'
rather than
        archive_dest = '/mnt/pgarch'
which is ambiguous...

> I thought about checking if this is a dir within the code. But
> this would make things too unfelxible.

Yes, otherwise the check would be there

> Second, we could make the user responsible of what he's doing by not
> giving him any target.

Remember, the user is specifying the archive_dest also, so the user is
completely responsible for how archiving actually occurs.

> Like you could then do things like:
> archive_program = 'gzip -d %s | tar rf /dev/nst0 - '

archive_program = 'gzip -d %s | tar rf %s - '

would be how I would use it in the example you give

> Which adds the file to a tar archive on his tape.
> If he wants to archive it on disk, let him do it this way:
> archive_program = 'cp %s /mnt/pgarch/'

archive_program = 'cp %s %s'

would be the way to specify that...

Thank you very much for feedback and your other contributions,

Best regards, Simon Riggs

---------------------------(end of broadcast)---------------------------
TIP 6: Have you searched our list archives?


Reply via email to