Re: Re: [COMMITTERS] pgsql: Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition b

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [COMMITTERS] pgsql: Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition b
Date: 2017-09-12 13:58:31
Message-ID: 20170912135831.jbknbivoosanexnx@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Robert Haas wrote:

> I just don't understand why you think there should be multiple
> spellings of the same bound. Generally, canonicalization is good.
> One of my fears here is that at least some people will get confused
> and think a bound from (minvalue, 0) to (maxvalue, 10) allows any
> value for the first column and a 0-9 value for the second column which
> is wrong.
>
> My other fear is that, since you've not only allowed this into the
> syntax but into the catalog, this will become a source of bugs for
> years to come. Every future patch that deals with partition bounds
> will now have to worry about testing
> unbounded-followed-by-non-unbounded. If we're not going to put back
> those error checks completely - and I think we should - we should at
> least canonicalize what actually gets stored.

Did anything happen on this, or did we just forget it completely?

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2017-09-12 14:02:53 pgsql: doc: Document default scope in LDAP URL
Previous Message Peter Eisentraut 2017-09-12 13:51:45 pgsql: Allow custom search filters to be configured for LDAP auth

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2017-09-12 13:58:37 Re: Automatic testing of patches in commit fest
Previous Message Peter Eisentraut 2017-09-12 13:55:51 Re: More flexible LDAP auth search filters?