Re: [sqlsmith] Crash in GetOldestSnapshot() - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [sqlsmith] Crash in GetOldestSnapshot()
Date
Msg-id CA+TgmoYkk2wVs5VWLWgM-4gCmCQsGb6q4pZ_8tKqfETe__AjXw@mail.gmail.com
Whole thread Raw
In response to Re: [sqlsmith] Crash in GetOldestSnapshot()  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [sqlsmith] Crash in GetOldestSnapshot()  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: [sqlsmith] Crash in GetOldestSnapshot()  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Sun, Aug 7, 2016 at 2:03 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> So I think in the short term what we should do about this is just fix
>> it so it doesn't crash.
>
> Well, we clearly need to fix GetOldestSnapshot so it won't crash,
> but I do not think that having RETURNING queries randomly returning
> "ERROR: no known snapshots" is acceptable even for a beta release.
> If we aren't prepared to do something about that before Monday,
> I think we need to revert 3e2f3c2e until we do have a fix for it.
>
> What I suggested just now in <2850.1470592623@sss.pgh.pa.us> might
> be implementable with a couple hours' work, though.  Do you have a
> reason to think it'd be insufficient?

No - if you can implement that quickly, I think it sounds like a great idea.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: [sqlsmith] Crash in GetOldestSnapshot()
Next
From: Robert Haas
Date:
Subject: Re: Slowness of extended protocol