Simon Riggs wrote:
> On 13 November 2017 at 12:55, Alvaro Herrera <alvherre@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
