]> granicus.if.org Git - postgresql/commit
Fix handling of empty arrays in array_fill().
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 5 Jan 2017 16:33:51 +0000 (11:33 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 5 Jan 2017 16:33:51 +0000 (11:33 -0500)
commit4e446563be720760efcb18e4a83b7189638f6ae8
treed1158a932dca0a0890e168475b16cb9f1b494297
parent696d40d303af1e92fbbe4192a93c5a94340fc22c
Fix handling of empty arrays in array_fill().

array_fill(..., array[0]) produced an empty array, which is probably
what users expect, but it was a one-dimensional zero-length array
which is not our standard representation of empty arrays.  Also, for
no very good reason, it rejected empty input arrays; that case should
be allowed and produce an empty output array.

In passing, remove the restriction that the input array(s) have lower
bound 1.  That seems rather pointless, and it would have needed extra
complexity to make the check deal with empty input arrays.

Per bug #14487 from Andrew Gierth.  It's been broken all along, so
back-patch to all supported branches.

Discussion: https://postgr.es/m/20170105152156.10135.64195@wrigleys.postgresql.org
src/backend/utils/adt/arrayfuncs.c
src/test/regress/expected/arrays.out
src/test/regress/sql/arrays.sql