Re: [HACKERS] Proposal: pg_rewind to skip config files - Mailing list pgsql-hackers

From Vladimir Borodin
Subject Re: [HACKERS] Proposal: pg_rewind to skip config files
Date
Msg-id C961CFCE-55C4-4F60-891D-CCE0CBA12BCF@simply.name
Whole thread Raw
In response to Re: [HACKERS] Proposal: pg_rewind to skip config files  (Chris Travers <chris.travers@adjust.com>)
Responses Re: [HACKERS] Proposal: pg_rewind to skip config files
Re: [HACKERS] Proposal: pg_rewind to skip config files
List pgsql-hackers

5 сент. 2017 г., в 12:31, Chris Travers <chris.travers@adjust.com> написал(а):

I think the simplest solution for now is to skip any files ending in .conf, .log, and serverlog.

Why don’t you want to solve the problem once? It is a bit harder to get consensus on a way how to do it, but it seems that there are no reasons to make temporary solution here.

For example, in archive_command we put WALs for archiving from pg_xlog/pg_wal into another directory inside PGDATA and than another cron task makes real archiving. This directory ideally should be skipped by pg_rewind, but it would not be handled by proposed change.


Long run, it would be nice to change pg_rewind from an opt-out approach to an approach of processing the subdirectories we know are important.

While it is definitely an awful idea the user can easily put something strange (i.e. logs) to any important directory in PGDATA (i.e. into base or pg_wal). Or how for example pg_replslot should be handled (I asked about it a couple of years ago [1])? It seems that a glob/regexp for things to skip is a more universal solution.



--
May the force be with you…

pgsql-hackers by date:

Previous
From: Jeevan Chalke
Date:
Subject: Re: [HACKERS] Replacing lfirst() with lfirst_node() appropriately in planner.c
Next
From: Konstantin Knizhnik
Date:
Subject: Re: [HACKERS] JIT compiling expressions/deform + inlining prototypev2.0