Re: [COMMITTERS] pgsql: Adjust OLDSERXID_MAX_PAGE based on BLCKSZ.

From: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
To: Robert Haas <rhaas(at)postgresql(dot)org>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Kevin Grittner <Kevin(dot)Grittner(at)wicourts(dot)gov>
Subject: Re: [COMMITTERS] pgsql: Adjust OLDSERXID_MAX_PAGE based on BLCKSZ.
Date: 2011-07-08 09:26:54
Message-ID: 4E16CD5E.9060303@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 07.07.2011 22:09, Robert Haas wrote:
> Adjust OLDSERXID_MAX_PAGE based on BLCKSZ.
>
> The value when BLCKSZ = 8192 is unchanged, but with larger-than-normal
> block sizes we might need to crank things back a bit, as we'll have
> more entries per page than normal in that case.

I'm getting a bunch of warnings on Windows related to this:

> .\src\backend\storage\lmgr\predicate.c(768): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(768): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(769): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(769): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(770): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(770): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(770): warning C4146: unary minus operator applied to unsigned type, result still unsigned
> .\src\backend\storage\lmgr\predicate.c(771): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(771): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(826): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(826): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(846): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(846): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(851): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(851): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(909): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(909): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(956): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(956): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(1036): warning C4307: '+' : integral constant overflow
> .\src\backend\storage\lmgr\predicate.c(1036): warning C4307: '+' : integral constant overflow

I'm not sure if these are harmless or not.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2011-07-08 14:15:19 pgsql: Add item about Clang support to 9.1 release notes
Previous Message Peter Eisentraut 2011-07-08 04:43:51 pgsql: Message style improvements

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2011-07-08 09:32:39 Re: WIP: Fast GiST index build
Previous Message Kyotaro HORIGUCHI 2011-07-08 09:21:16 Re: make_greater_string() does not return a string in some cases