Re: Small fix for inv_getsize

Denis Perchine <dyp@perchine.com> writes:
> Just realized that inv_getsize is a little bit wrong :-)). It just get the
> first page, not last
> Here is the patch which will fix the behavior.

No it doesn't; it's a loop, and your patch will change nothing.  Your
original version tried to stop after fetching one tuple, which was
wrong because of visibility considerations.

Now that I think about it, this code could do a two-key scan backwards
and stop after finding the first (last) valid tuple, but that's more
than a one-line change.

            regards, tom lane