On Sun, Nov 15, 2015 at 7:20 PM, and...@anarazel.de <and...@anarazel.de> wrote:
>> + * We reserve a few predefined tranche IDs. These values will never be
>> + * returned by LWLockNewTrancheId.
>> + */
>> +#define LWTRANCHE_MAIN 0
>> +#define LWTRANCHE_BUFFER_CONTENT 1
>> +#define LWTRANCHE_BUFFER_IO_IN_PROGRESS 2
>> +#define LWTRANCHE_LAST_BUILTIN_ID
> Nitpick: I'm inclined to use an enum to avoid having to adjust the last
> builtin id when adding a new builtin tranche.
I prefer to do it this way because sometimes enums require a cast.
But if you do the work, I'm not going to fight you over this.
(If I do the work, on the other hand, ...)
> Looks mis-indented now, similarly in a bunch of other places. Maybe
> pg-indent afterwards?
pgindent doesn't change anything for me.
> So, looks good to me.
The Enterprise PostgreSQL Company
Sent via pgsql-hackers mailing list (email@example.com)
To make changes to your subscription: