On Sat, Jan 22, 2011 at 12:33 AM, Robert Haas <robertmh...@gmail.com> wrote:
> On Sat, Jan 15, 2011 at 11:52 AM, Greg Smith <g...@2ndquadrant.com> wrote:
>> Where I was expecting that setting to be "4" instead for 32kB. So there's
>> probably some minor bug left in where I inserted this into the
>> initialization sequence.
>
> So I exposed the relevant formatting logic from guc.c as a separate function
i have read this very breafly, so not much comment... just a few questions...
why is this better than using XLOG_BUFFER_MIN? (the same for the 8
buffers assigned just above of it)
+ else if (XLOGbuffers < 4)
+ XLOGbuffers = 4;
also this
+ Assert(XLOGbuffers > 0);
maybe should be
Assert(XLOGbuffers >= XLOG_BUFFER_MIN);
while you move the code, why didn't you keep this comment?
- /*
- * Use int64 arithmetic to avoid
overflows in units
- * conversion.
- */
--
Jaime Casanova www.2ndQuadrant.com
Professional PostgreSQL: Soporte y capacitación de PostgreSQL
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
