pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter@eisentraut.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix a warning in Perl test code
Date: Wed, 27 Dec 2023 16:23:59 +0000
Message-ID: <E1rIWhK-00CB9I-KZ@gemulon.postgresql.org> (raw)

Fix a warning in Perl test code

The code was passing a scalar argument to node->restart(), but it was
expecting a hash, which causes a warning from Perl ("Odd number of
elements in hash assignment").

But the node->restart() function doesn't take a mode argument anyway.
This was probably copied from an incorrect comment (see commit
750c59d7ec).  The default restart mode is already "fast", so the test
should still be semantically correct without explicitly specifying the
mode.

Discussion: https://www.postgresql.org/message-id/e3f4bf1b-63d3-408a-b07e-d35a0fdf1b98@eisentraut.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a7ebd82b9e50960cac59868fa1dd3cbe12de8c7b

Modified Files
--------------
src/test/recovery/t/006_logical_decoding.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)



Message-ID: <E1rIWhK-00CB9I-KZ@gemulon.postgresql.org>
Permalink:  ../E1rIWhK-00CB9I-KZ@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1rIWhK-00CB9I-KZ@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: peter@eisentraut.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix a warning in Perl test code
  In-Reply-To: <E1rIWhK-00CB9I-KZ@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox