Re: pgsql: Allow HOT updates for some expression indexes - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Allow HOT updates for some expression indexes
Date
Msg-id 11787.1522184835@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Allow HOT updates for some expression indexes  (Simon Riggs <simon@2ndquadrant.com>)
Responses Re: pgsql: Allow HOT updates for some expression indexes  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
Simon Riggs <simon@2ndquadrant.com> writes:
> On 27 March 2018 at 20:11, Simon Riggs <simon@2ndquadrant.com> wrote:
>> Buildfarm failure seen, investigating

> Error on FreeBSD only , perhaps timing-related as a result of using
> pg_stat_get_xact_tuples_hot_updated()
> in tests.

It's by now obvious that it's not just "FreeBSD only".

The test script appears to imagine that pgstats outputs update
instantaneously.  I'm rather astonished that it passes anywhere,
much less that it seemingly passes reliably enough for you
to have not been noticed in precommit testing.

The existing "stats" test moves mountains to get repeatable results,
and is pretty slow in consequence, and even so it doesn't always succeed.
I think that rather than going down that path, you should drop this test
approach and think of some other way.

            regards, tom lane


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Allow memory contexts to have both fixed and variable identstri
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Allow HOT updates for some expression indexes