Simon Riggs wrote: > On 13 November 2017 at 12:55, Alvaro Herrera <alvhe...@alvh.no-ip.org> wrote: > > Somehow I managed to include an unrelated patch as attachment. Here's > > another attempt (on which I also lightly touched ddl.sgml with relevant > > changes). > > Looks good. Some minor comments below. > > 0001- Simplify > Seems useful as separate step; agree with everything, no further comments
Thanks, pushed. > Why uint16? Why not just uint? *Shrug*. 16 bits seem plenty enough. I changed it to bits16, which we use in other places for bitmasks. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
