Alvaro Herrera  wrote:
 
> We don't support BLCKSZ higher than 32k anyway
 
Thanks for pointing that out.  Then I think we should declare sum1 to
be uint and sum2 to be uint64.  We can take out the "% 255" out from
where it sits in the v2 patch, and just add something like this after
the sums are generated:
 
    sum1 %= 255;
    sum2 %= 255;
 
Or, of course, we can just do it on the line where we combine the two
sums for the final result, if that's not too hard to read.
 
-Kevin

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to