I wrote:
> Peter Geoghegan <p...@heroku.com> writes:
>> I think we should change it. It seems like a bug to me.

> Me too.  Is it enough bug-like to be something to back-patch, or should
> we just change it in HEAD?

Actually, there's a significantly worse bug here: I just realized that the
page type tests are done in the wrong order.  A deleted page that was
formerly a leaf will be reported as though it was a live leaf page,
because both the BTP_LEAF and BTP_DELETED flags are set for such a page.

It looks like this was done correctly to begin with, and I broke it in
d287818eb514d431b1a68e1f3940cd958f82aa34.  Not sure what I was thinking :-(

Anyway, I think that puts the final nail in the coffin of the idea that
the current code's behavior is sane enough to preserve.  I think we should
fix all these things and back-patch 'em all.

                        regards, tom lane

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to