Horton

Re: pgsql: psql: add an optional execution-count limit to \watch.

Tom Lane <tgl@sss.pgh.pa.us>

View thread View on postgr.es
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Aleksander Alekseev <aleksander@timescale.com>
Cc: pgsql-committers@lists.postgresql.org, Alexander Korotkov <aekorotkov@gmail.com>, Andrew Dunstan <andrew@dunslane.net>
Date: 2023-04-18T17:34:12Z
Lists: pgsql-hackers 
Aleksander Alekseev <aleksander@timescale.com> writes:
>> The attached makes check-world pass for me under these conflicting
>> environment settings, but it's kind of a scary change.  Thoughts?

> FWIW my MacOS and Linux laptops have no complaints about the patch.

I realized that we don't actually need to "use locale" in Utils.pm
itself for this to work, which greatly assuages my fears of unexpected
side-effects.  Pushed that way; I shall now retire to a safe distance
and watch the buildfarm.

			regards, tom lane

Commits

  1. Fix Utils.pm's locale-munging so that Perl itself is also affected.

  2. Fix locale-dependent test case.

  3. psql: add an optional execution-count limit to \watch.