Fix handling of empty arrays in array_fill().
authorTom Lane
Thu, 5 Jan 2017 16:33:51 +0000 (11:33 -0500)
committerTom Lane
Thu, 5 Jan 2017 16:33:51 +0000 (11:33 -0500)
commit4555a375a1b27a49f5f9da2474159557360d17de
tree1e8d6ee99504aeba5400f192f6779cf0dde228c7
parent50c8196f946d55f7f4b998491e48f592f711c4fe
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[email protected]
src/backend/utils/adt/arrayfuncs.c
src/test/regress/expected/arrays.out
src/test/regress/sql/arrays.sql