pgsql: Rename pgbench min/max to least/greatest, and fix handling of do

Started by Tom Laneabout 10 years ago3 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

Rename pgbench min/max to least/greatest, and fix handling of double args.

These functions behave like the backend's least/greatest functions,
not like min/max, so the originally-chosen names invite confusion.
Per discussion, rename to least/greatest.

I also took it upon myself to make them return double if any input is
double. The previous behavior of silently coercing all inputs to int
surely does not meet the principle of least astonishment.

Copy-edit some of the other new functions' documentation, too.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/7a622b2731db5d0f6db8a3d0af88177f96d1cb2e

Modified Files
--------------
doc/src/sgml/ref/pgbench.sgml | 25 +++++++++--------
src/bin/pgbench/exprparse.y | 8 +++---
src/bin/pgbench/pgbench.c | 64 ++++++++++++++++++++++++++++++++-----------
src/bin/pgbench/pgbench.h | 4 +--
4 files changed, 67 insertions(+), 34 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Fabien COELHO
coelho@cri.ensmp.fr
In reply to: Tom Lane (#1)
Re: pgsql: Rename pgbench min/max to least/greatest, and fix handling of do

Rename pgbench min/max to least/greatest, and fix handling of double
args.

Ok.

The very minor patch attached re-establishes the alphabetical order when
listing functions names in pgbench documentation.

--
Fabien.

Attachments:

pgbench-func-abc-1.patchtext/x-diff; name=pgbench-func-abc-1.patchDownload+7-7
#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Fabien COELHO (#2)
Re: pgsql: Rename pgbench min/max to least/greatest, and fix handling of do

The very minor patch attached re-establishes the alphabetical order when
listing functions names in pgbench documentation.

Pushed, thanks.

regards, tom lane

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers