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
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 |