Re: [COMMITTERS] pgsql: Sigh ... - Mailing list pgsql-patches

From Tom Lane
Subject Re: [COMMITTERS] pgsql: Sigh ...
Date
Msg-id 26232.1209767321@sss.pgh.pa.us
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Sigh ...  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: [COMMITTERS] pgsql: Sigh ...
List pgsql-patches
Andrew Dunstan <andrew@dunslane.net> writes:
> !         print O "#define RELSEG_SIZE ",
> !             (1024 / $self->{options}->{blocksize}) *
> !                 $self->{options}->{segsize} * 1024, "\n";

This doesn't look quite right; unless the arithmetic is being done in
floating point?  I had it like this in configure.in:

RELSEG_SIZE=`expr '(' 1024 '*' ${segsize} / ${blocksize} ')' '*' 1024`

Also it looks like you missed adding segsize to the config.pl comments.

            regards, tom lane

pgsql-patches by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: [COMMITTERS] pgsql: Sigh ...
Next
From: Tom Lane
Date:
Subject: Re: plpgsql CASE statement - last version