Re: [HACKERS] WIP: Covering + unique indexes. - Mailing list pgsql-hackers

From Andrey Borodin
Subject Re: [HACKERS] WIP: Covering + unique indexes.
Date
Msg-id 7458A9E7-038F-47EF-9CE1-41E64BC117B0@yandex-team.ru
Whole thread Raw
In response to Re: [HACKERS] WIP: Covering + unique indexes.  (Peter Geoghegan <pg@bowt.ie>)
List pgsql-hackers
> 29 нояб. 2017 г., в 8:45, Peter Geoghegan <pg@bowt.ie> написал(а):
>
> On Tue, Nov 28, 2017 at 6:16 PM, Michael Paquier
> <michael.paquier@gmail.com> wrote:
>> On Sun, Nov 12, 2017 at 8:40 PM, Andrey Borodin <x4mmm@yandex-team.ru> wrote:
>>> Postgres crashes:
>>> TRAP: FailedAssertion("!(((const void*)(&isNull) != ((void*)0)) && (scankey->sk_attno) > 0)", File: "nbtsearch.c",
Line:466) 
>>>
>>> May be I'm doing something wrong or amcheck support will go with different patch?
>>
>> Usually amcheck complaining is a sign of other symptoms. I am marking
>> this patch as returned with feedback for now as no updates have been
>> provided after two weeks.
>
> It looks like amcheck needs to be patched -- a simple oversight.
> amcheck is probably calling _bt_compare() without realizing that
> internal pages don't have the extra attributes (just leaf pages,
> although they should also not participate in comparisons in respect of
> included/extra columns). There were changes to amcheck at one point in
> the past. That must have slipped through again. I don't think it's
> that complicated.
>


There is no doubts that this will be fixed. Therefor I propose move to next CF with Waiting for Author status.

Best regards, Andrey Borodin.

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [HACKERS] Re: [COMMITTERS] pgsql: Remove pgbench "progress" testpending solution of its timing is (fwd)
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] proposal - Default namespaces for XPath expressions(PostgreSQL 11)