Re: Disallow arrays with non-standard lower bounds

From: Craig Ringer <craig(at)2ndquadrant(dot)com>
To: David Fetter <david(at)fetter(dot)org>, Jim Nasby <jim(at)nasby(dot)net>
Cc: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Disallow arrays with non-standard lower bounds
Date: 2014-01-12 10:38:24
Message-ID: 52D270A0.6020501@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 01/10/2014 07:41 AM, David Fetter wrote:
> On Thu, Jan 09, 2014 at 04:30:25PM -0600, Jim Nasby wrote:
>> ISTM that allowing users to pick arbitrary lower array bounds was a
>> huge mistake. I've never seen anyone make use of it, can't think of
>> any legitimate use cases for it, and hate the stupendous amount of
>> extra code needed to deal with it.
>>
>> Obviously we can't just drop support, but what about an initdb (or
>> hell, even configure) option to disallow arrays with a lower bound
>> <> 1? Unfortunately we can't do this with a GUC since you can store
>> arrays in a table.
>
> We have dropped support, as you put it, for bigger and harder-hitting
> mistakes than this.

Implicit casts to text, anybody?

Everybody used them. The project dropped them anyway, and did so pretty
much unconditionally at that.

This is a vastly less heavily used feature. I'd be really, really glad
to see it disabled by default at initdb time, with a warning saying the
initdb option to enable it will go away in a few versions.

--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2014-01-12 12:16:15 Fwd: patch: make_timestamp function
Previous Message Craig Ringer 2014-01-12 10:12:01 Re: WIP patch (v2) for updatable security barrier views