On Fri, Jul 4, 2008 at 1:01 PM, Zdenek Kotala <[EMAIL PROTECTED]> wrote:
>
>
> Good catch. I lost in basic arithmetic. What I see now that original
> definition count sizeof(ItemIdData) twice and on other side it does not take
> care about MAXALING correctly. I think correct formula is:
>
> #define HashMaxItemSize(page) \
>        (PageGetPageSize(page) - \
>          ( MAXALIGN(SizeOfPageHeaderData + sizeof(ItemIdData))+ \
>            MAXALIGN(sizeof(HashPageOpaqueData)) \
>          )\
>         )
>
> What do you think?
>


Yes. I think that's the correct way.


Thanks,
Pavan

-- 
Pavan Deolasee
EnterpriseDB http://www.enterprisedb.com

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

Reply via email to