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

From: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
To: Stephen Frost <sfrost(at)snowman(dot)net>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Noah Misch <noah(at)leadboat(dot)com>, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, "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-14 16:59:46
Message-ID: CAKFQuwZdmLNM8hKM7Heuf5__VKeTYmNk7QkfqGxpq_0a9eXQGw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, Sep 14, 2017 at 8:41 AM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:

> Robert, all,
>
> * Robert Haas (robertmhaas(at)gmail(dot)com) wrote:
>
> > >
> > > I vote for rejecting it. DDL compatibility is less valuable than other
> > > compatibility. The hypothetical affected application can change its
> DDL to
> > > placate PostgreSQL and use that modified DDL for all other databases,
> too.
> >
> > OK. Any other votes?
>
> I haven't been as close to this as others, so perhaps my vote is only
> 0.5 on this specific case, but that's my feeling on it.
>

I think we are being consistent as a project by enforcing strictness of
input in this situation so I'll toss my +0.5/+1​ here as well.

David J.

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2017-09-14 17:28:02 pgsql: Avoid use of bool in thread_test.c
Previous Message Robert Haas 2017-09-14 16:36:57 pgsql: Make RelationGetPartitionDispatchInfo expand depth-first.

Browse pgsql-hackers by date

  From Date Subject
Next Message Jesper Pedersen 2017-09-14 17:07:11 Re: [POC] hash partitioning
Previous Message Peter Eisentraut 2017-09-14 16:58:44 Re: [JDBC] Channel binding support for SCRAM-SHA-256