Re: Small fix for inv_getsize

From: Denis Perchine <dyp(at)perchine(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Small fix for inv_getsize
Date: 2000-11-02 17:02:23
Message-ID: 0011022302230F.31936@dyp.perchine.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

> > 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.

Actual logic is to find the maximum of pageno, for specified oid.
I do index scan on 2-keys index, specifying only one key as constraint...
If I do index scan forward I will get the smallest pageno first...
Otherwise I get the highest pageno... And this is what I want...
Or I get something wrong? Isn't this how order by on index is done?

--
Sincerely Yours,
Denis Perchine

----------------------------------
E-Mail: dyp(at)perchine(dot)com
HomePage: http://www.perchine.com/dyp/
FidoNet: 2:5000/120.5
----------------------------------

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2000-11-02 17:32:38 Re: Small fix for inv_getsize
Previous Message Tom Lane 2000-11-02 16:49:38 Re: Small fix for inv_getsize