Re: TAP / recovery-test fs-level backups, psql enhancements etc

Поиск
Список
Период
Сортировка
От Craig Ringer
Тема Re: TAP / recovery-test fs-level backups, psql enhancements etc
Дата
Msg-id CAMsr+YG6-NUqH3CbexQZ0nkVM+QpGJGV6yhJVAH4Ssdd61c2xA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: TAP / recovery-test fs-level backups, psql enhancements etc  (Michael Paquier <michael.paquier@gmail.com>)
Ответы Re: TAP / recovery-test fs-level backups, psql enhancements etc  (Michael Paquier <michael.paquier@gmail.com>)
Re: TAP / recovery-test fs-level backups, psql enhancements etc  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Re: TAP / recovery-test fs-level backups, psql enhancements etc  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Re: TAP / recovery-test fs-level backups, psql enhancements etc  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Список pgsql-hackers
Дерево обсуждения
On 3 March 2016 at 13:23, Michael Paquier <michael.paquier@gmail.com> wrote:
On Thu, Mar 3, 2016 at 2:20 PM, Craig Ringer <craig@2ndquadrant.com> wrote:
> On the Perl 5.8.8 test env I've set up now, per
>
> http://www.postgresql.org/message-id/CAMsr+YGR6pU-gUyp-FT98XwXAsc9n6j-awZAqxvW_+P3RTC7cg@mail.gmail.com
>
> master currently fails with
>
> t/004_timeline_switch....."remove_tree" is not exported by the File::Path
> module
>
> remove_tree is only in File::Path 2.07 but Perl 5.8.8 has 1.08. No buildfarm
> member has complained, so clearly we don't actually test with the stated
> supported Perl version.
>
> The attached patch fixes it to use the legacy File::Path interface 'rmtree'
> so master runs on 588 again.

Crap. Thanks for spotting that, I was careless. You could still use
qw() and not use File::Path::rmtree, but that's a matter of taste.

New patch series.

The first three are simple fixes that should go in without fuss:

001 fixes the above 5.8.8 compat issue.

002  fixes another minor whoopsie, a syntax error in  src/test/recovery/t/003_recovery_targets.pl that never got noticed because exit codes are ignored.

003 runs perltidy on PostgresNode.pm to bring it back into conformance after the recovery tests commit.

The rest are feature patches:

004 allows filtering on RecursiveCopy by a predicate function. Needed for filesystem level backups (007). It could probably be squashed with 007 if desired.

005 adds the new psql functions psql_expert and psql_check. Then 006 renames psql_expert to psql and fixes up all call sites across the tree to use the new interface. I found the bug in 002 as part of that process. I anticipate that 005 and 006 would be squashed into one commit to master, but I've kept them separate in my tree for easier review.

007 adds PostgresNode support for hot and cold filesystem-level backups using pg_start_backup and pg_stop_backup, which will be required for some coming tests and are useful by themselves.


Each patch is pretty simple except for the psql patch, and even that's not exactly hairy stuff. The potential risks are low as this is code that's not part of the server and isn't even run on 'make check' by default. I've tested it all under a real Perl 5.8.8 on CentOS 5.

I don't expect to be doing much more on the framework at this point as I want to be able to get back to the code I had to enhance the framework in order to test....

--
 Craig Ringer                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services
Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: Proposal: "Causal reads" mode for load balancing reads without stale data
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: Re: redo failed in physical streaming replication while stopping the master server