Re: Allow default \watch interval in psql to be configured - Mailing list pgsql-hackers
From | Pavel Stehule |
---|---|
Subject | Re: Allow default \watch interval in psql to be configured |
Date | |
Msg-id | CAFj8pRAjjk4yuXMi-uUbAwmrBygwQT6Bwuy=UZu8JqCebzvetQ@mail.gmail.com Whole thread Raw |
In response to | Re: Allow default \watch interval in psql to be configured (Pavel Stehule <pavel.stehule@gmail.com>) |
Responses |
Re: Allow default \watch interval in psql to be configured
|
List | pgsql-hackers |
st 26. 3. 2025 v 7:59 odesílatel Pavel Stehule <pavel.stehule@gmail.com> napsal:
Hiút 25. 3. 2025 v 20:09 odesílatel Daniel Gustafsson <daniel@yesql.se> napsal:> On 24 Mar 2025, at 13:42, Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> wrote:
> LGTM. I think this is RFC. Updated CF entry.
Thanks all for review, committed.regress tests fails now in mymake[2]: Vstupuje se do adresáře „/home/pavel/src/postgresql/src/bin/psql“
echo "# +++ tap check in src/bin/psql +++" && rm -rf '/home/pavel/src/postgresql/src/bin/psql'/tmp_check && /usr/bin/mkdir -p '/home/pavel/src/postgresql/src/bin/psql'/tmp_check && cd . && TESTLOGDIR='/home/pavel/src/postgresql/src/bin/psql/tmp_check/log' TESTDATADIR='/home/pavel/src/postgresql/src/bin/psql/tmp_check' PATH="/home/pavel/src/postgresql/tmp_install/usr/local/pgsql/master/bin:/home/pavel/src/postgresql/src/bin/psql:$PATH" LD_LIBRARY_PATH="/home/pavel/src/postgresql/tmp_install/usr/local/pgsql/master/lib" INITDB_TEMPLATE='/home/pavel/src/postgresql'/tmp_install/initdb-template PGPORT='65432' top_builddir='/home/pavel/src/postgresql/src/bin/psql/../../..' PG_REGRESS='/home/pavel/src/postgresql/src/bin/psql/../../../src/test/regress/pg_regress' share_contrib_dir='/home/pavel/src/postgresql/tmp_install/usr/local/pgsql/master/share/' /usr/bin/prove -I ../../../src/test/perl/ -I . t/*.pl
# +++ tap check in src/bin/psql +++
t/001_basic.pl ........... 46/?
# Failed test 'WATCH_INTERVAL variable is set and updated: exit code 0'
# at t/001_basic.pl line 436.
# got: '3'
# expected: '0'
# Failed test 'WATCH_INTERVAL variable is set and updated: no stderr'
# at t/001_basic.pl line 436.
# got: 'psql:<stdin>:2: error: "0.001" is out of range for "WATCH_INTERVAL"'
# expected: ''
# Failed test 'WATCH_INTERVAL variable is set and updated: matches'
# at t/001_basic.pl line 436.
# '2'
# doesn't match '(?^lm:^2$
# ^0.001$
# ^2$)'
# Looks like you failed 3 tests of 116.
t/001_basic.pl ........... Dubious, test returned 3 (wstat 768, 0x300)
Failed 3/116 subtests
t/010_tab_completion.pl .. ok
t/020_cancel.pl .......... ok
Test Summary Report
-------------------
t/001_basic.pl (Wstat: 768 (exited 3) Tests: 116 Failed: 3)
Failed tests: 95-97
Non-zero exit status: 3
Files=3, Tests=207, 7 wallclock secs ( 0.12 usr 0.02 sys + 3.38 cusr 1.72 csys = 5.24 CPU)
Result: FAILThe reason is probably my LANG=cs_CZ.UTF8. When I switched to LANG=C, then tests passed.
The main problem is in numeric format. Czech uses the comma instead of the dot.
pavel@nemesis:~/src/postgresql/src/bin/psql$ LANG=C psql -c "\set WATCH_INTERVAL 0.001"
pavel@nemesis:~/src/postgresql/src/bin/psql$ LANG=cs_CZ.UTF8 psql -c "\set WATCH_INTERVAL 0.001"
"0.001" is out of range for "WATCH_INTERVAL"
pavel@nemesis:~/src/postgresql/src/bin/psql$ LANG=cs_CZ.UTF8 psql -c "\set WATCH_INTERVAL 0,001"
pavel@nemesis:~/src/postgresql/src/bin/psql$ LANG=cs_CZ.UTF8 psql -c "\set WATCH_INTERVAL 0.001"
"0.001" is out of range for "WATCH_INTERVAL"
pavel@nemesis:~/src/postgresql/src/bin/psql$ LANG=cs_CZ.UTF8 psql -c "\set WATCH_INTERVAL 0,001"
Regards
Pavel
RegardsPavel--
Daniel Gustafsson
pgsql-hackers by date: