On 2020-09-09 12:41 a.m., gkokolatos@pm.me wrote:
> ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
> On Tuesday, 8 September 2020 22:26, David Zhang <david.zhang@highgo.ca> wrote:
>
>>
>> I found the function "table_relation_size" is only used by buffer
>> manager for "RELKIND_RELATION", "RELKIND_TOASTVALUE" and
>> "RELKIND_MATVIEW", i.e.
>>
>> case RELKIND_RELATION:
>> case RELKIND_TOASTVALUE:
>> case RELKIND_MATVIEW:
>> {
>> /*
>> * Not every table AM uses BLCKSZ wide fixed size blocks.
>> * Therefore tableam returns the size in bytes - but
>> for the
>> * purpose of this routine, we want the number of blocks.
>> * Therefore divide, rounding up.
>> */
>> uint64 szbytes;
>>
>> szbytes = table_relation_size(relation, forkNum);
>>
>> return (szbytes + (BLCKSZ - 1)) / BLCKSZ;
>> }
>>
>> So using "calculate_relation_size" and "calculate_toast_table_size" in
>> "calculate_table_size" is easy to understand and the original logic is
>> simple.
>>
> You are correct. This is the logic that is attempted to be applied
> in dbsize.c in this patch.
>
> So what do you think of the patch?
I would suggest to keep the original logic unless there is a bug.
--
David
Software Engineer
Highgo Software Inc. (Canada)
www.highgo.ca
