diff options
| author | Andrew Dunstan <andrew@dunslane.net> | 2021-07-29 05:58:08 -0400 |
|---|---|---|
| committer | Andrew Dunstan <andrew@dunslane.net> | 2021-07-29 05:58:08 -0400 |
| commit | 201a76183e2056c2217129e12d68c25ec9c559c8 (patch) | |
| tree | f25c8d8146f9a3970f776e4ac01f63af4f462184 /src/bin/pg_rewind/t/RewindTest.pm | |
| parent | dbfe6e4b1766369654e20113b0cab79bd939d544 (diff) | |
| download | postgresql-201a76183e2056c2217129e12d68c25ec9c559c8.tar.gz postgresql-201a76183e2056c2217129e12d68c25ec9c559c8.zip | |
Unify PostgresNode's new() and get_new_node() methods
There is only one constructor now for PostgresNode, with the idiomatic
name 'new'. The method is not exported by the class, and must be called
as "PostgresNode->new('name',[args])". All the TAP tests that use
PostgresNode are modified accordingly. Third party scripts will need
adjusting, which is a fairly mechanical process (I just used a sed
script).
Diffstat (limited to 'src/bin/pg_rewind/t/RewindTest.pm')
| -rw-r--r-- | src/bin/pg_rewind/t/RewindTest.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/bin/pg_rewind/t/RewindTest.pm b/src/bin/pg_rewind/t/RewindTest.pm index 938c661afc2..367b99a438f 100644 --- a/src/bin/pg_rewind/t/RewindTest.pm +++ b/src/bin/pg_rewind/t/RewindTest.pm @@ -128,7 +128,7 @@ sub setup_cluster # Initialize primary, data checksums are mandatory $node_primary = - get_new_node('primary' . ($extra_name ? "_${extra_name}" : '')); + PostgresNode->new('primary' . ($extra_name ? "_${extra_name}" : '')); # Set up pg_hba.conf and pg_ident.conf for the role running # pg_rewind. This role is used for all the tests, and has @@ -176,7 +176,7 @@ sub create_standby my $extra_name = shift; $node_standby = - get_new_node('standby' . ($extra_name ? "_${extra_name}" : '')); + PostgresNode->new('standby' . ($extra_name ? "_${extra_name}" : '')); $node_primary->backup('my_backup'); $node_standby->init_from_backup($node_primary, 'my_backup'); my $connstr_primary = $node_primary->connstr(); |