Re: Re: [COMMITTERS] pgsql: Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition b
Alvaro Herrera <alvherre@alvh.no-ip.org>
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Dean Rasheed <dean.a.rasheed@gmail.com>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2017-09-12T13:58:31Z
Lists: 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
Commits
-
After a MINVALUE/MAXVALUE bound, allow only more of the same.
- e8b65986ba0d 10.0 landed
- 9361f6f54e3f 11.0 landed
-
Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition bounds.
- d363d42bb9a4 10.0 cited