On 12.06.25 08:26, jian he wrote:
in contrib/amcheck/verify_heapam.c, check_tuple
report_corruption(ctx,
psprintf("number of attributes %u exceeds
maximum expected for table %u",
ctx->natts,
RelationGetDescr(ctx->rel)->natts));
Agreed this is misleading.
i think it should be
report_corruption(ctx,
psprintf("number of attributes %u exceeds
maximum expected for table %u",
ctx->natts,
RelationGetRelid(ctx->rel)));
or we can rephrase it another way, also mentioning
``RelationGetDescr(ctx->rel)->natts``.
I think they did want to mention RelationGetDescr(ctx->rel)->natts. How
about
"number of attributes %u exceeds maximum expected for table (%u)"
