Re: Bug in nbtree optimization to skip > operator comparisons (or < comparisons in backwards scans)

Mon, 11 Dec 2023 11:00:05 -0800 

On Mon, Dec 11, 2023 at 5:56 PM Alexander Korotkov <aekorot...@gmail.com> wrote:
> > BTW, do we really need to keep around the BTScanOpaqueData.firstPage
> > field? Why can't the call to _bt_readpage from _bt_first (and from
> > _bt_endpoint) just pass "firstPage=true" as a simple argument? Note
> > that the first call to _bt_readpage must take place from _bt_first (or
> > from _bt_endpoint). The first _bt_first call is already kind of
> > special, in a way that is directly related to this issue. I added some
> > comments about that to today's commit c9c0589fda, in fact -- I think
> > it's an important issue in general.
>
> Please, check the attached patchset.

Sorry, I forgot the attachment.  Here it is.

------
Regards,
Alexander Korotkov

Attachment: 0001-Remove-BTScanOpaqueData.firstPage-v2.patch
Description: Binary data

Attachment: 0002-Fix-requiredOppositeDir-bug-v2.patch
Description: Binary data

Reply via email to