Re: [PATCH] Improve amcheck to also check UNIQUE constraint in btree index.

Поиск
Список
Период
Сортировка
От Mark Dilger
Тема Re: [PATCH] Improve amcheck to also check UNIQUE constraint in btree index.
Дата
Msg-id 65DAA5C7-457C-4BF8-ACAF-43E931A67BA0@enterprisedb.com
обсуждение исходный текст
Ответ на Re: [PATCH] Improve amcheck to also check UNIQUE constraint in btree index.  (Mark Dilger <mark.dilger@enterprisedb.com>)
Ответы Re: [PATCH] Improve amcheck to also check UNIQUE constraint in btree index.
Список pgsql-hackers

> On May 17, 2024, at 12:10 PM, Mark Dilger <mark.dilger@enterprisedb.com> wrote:
>
>> Amcheck with checkunique option does check uniqueness violation between pages. But it doesn't warranty detection of
crosspage uniqueness violations in extremely rare cases when the first equal index entry on the next page corresponds
totuple that is not visible (e.g. dead). In this, I followed the Peter's notion [1] that checking across a number of
deadequal entries that could theoretically span even across many pages is an unneeded code complication and amcheck is
nota tool that provides any warranty when checking an index. 
>
> This confuses me a bit.  The regression test creates a table and index but never performs any DELETE nor any UPDATE
operations,so none of the index entries should be dead.  If I am understanding you correct, I'd be forced to conclude
thatthe uniqueness checking code is broken.  Can you take a look? 

On further review, the test was not anticipating the error message "high key invariant violated for index".  That
wasn'tseen in calls to bt_index_parent_check(), but appears as one of the errors from bt_index_check().  I am rerunning
thetest now.... 

—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company






В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tomas Vondra
Дата:
Сообщение: allow sorted builds for btree_gist
Следующее
От: Pavel Borisov
Дата:
Сообщение: Re: [PATCH] Improve amcheck to also check UNIQUE constraint in btree index.