pgsql: Improve handling of psql \watch's interval argument - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Improve handling of psql \watch's interval argument
Date
Msg-id E1pcbYg-003jli-Fg@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Improve handling of psql \watch's interval argument

A failure in parsing the interval value defined in the \watch command
was silently switched to 1s of interval between two queries, which can
be confusing.  This commit improves the error handling, and a couple of
tests are added to check after:
- An incorrect value.
- An out-of-range value.
- A negative value.

A value of zero is able to work now, meaning that there is no interval
of time between two queries in a \watch loop.  No backpatch is done, as
it could break existing applications.

Author: Andrey Borodin
Reviewed-by: Kyotaro Horiguchi, Nathan Bossart, Michael Paquier
Discussion: https://postgr.es/m/CAAhFRxiZ2-n_L1ErMm9AZjgmUK=qS6VHb+0SaMn8sqqbhF7How@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6f9ee74d45aa7c1836b6c12b6e2a958327c16ab5

Modified Files
--------------
src/bin/psql/command.c      | 18 +++++++++++++++---
src/bin/psql/t/001_basic.pl | 17 +++++++++++++++++
2 files changed, 32 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: pgsql: MSVC: Don't build ldap_password_func if not building with ldap
Next
From: Michael Paquier
Date:
Subject: pgsql: Add .gitignore to ldap_password_func