Re: [COMMITTERS] pgsql: Remove secondary checkpoint - Mailing list pgsql-committers

From Simon Riggs
Subject Re: [COMMITTERS] pgsql: Remove secondary checkpoint
Date
Msg-id CANP8+jK_MsDa8dFV5wO3=U2V6b3oHzQ55D5nqd-o9Yu31r-rGQ@mail.gmail.com
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Remove secondary checkpoint  (Amit Kapila <amit.kapila16@gmail.com>)
Responses Re: [COMMITTERS] pgsql: Remove secondary checkpoint  (Andres Freund <andres@anarazel.de>)
List pgsql-committers
On 20 November 2017 at 08:38, Amit Kapila <amit.kapila16@gmail.com> wrote:
> On Sat, Nov 11, 2017 at 10:58 PM, Andreas Seltenreich
> <seltenreich@gmx.de> wrote:
>> Hi,
>>
>> sqlsmith doesn't like commit 4b0d28de06:
>>
>> ,----
>> | regression=> select * from pg_control_checkpoint();
>> | server closed the connection unexpectedly
>> | TRAP: FailedAssertion("!((atti->attalign) == 's')", File: "heaptuple.c", Line: 126)
>> `----
>>
>> On a build with assertions disabled, the statement fails with an error
>> instead:
>>
>> ,----
>> | regression=> select * from pg_control_checkpoint();
>> | ERROR:  function return row and query-specified return row do not match
>> | DETAIL:  Returned row contains 19 attributes, but query expects 18.
>> `----
>>
>> The attached patch fixes it for me.
>>
>
>
> Your patch looks correct to me.  I can reproduce the problem and
> verified that patch fixes the problem.  It is better to track this in
> CF if not already tracked.

What email and patch is this referring to?

-- 
Simon Riggs                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: [HACKERS] [COMMITTERS] pgsql: Fix freezing of a dead HOT-updatedtuple
Next
From: Andres Freund
Date:
Subject: Re: [COMMITTERS] pgsql: Remove secondary checkpoint