From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Peter Geoghegan <pg(at)bowt(dot)ie>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pgsql: Compute XID horizon for page level index vacuum on primary. |
Date: | 2019-03-30 10:32:22 |
Message-ID: | CA+hUKGJ1NA0zoB5PTaxLg_FM1S0FVkdbFxjGiYa+syp48g9=mQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Fri, Mar 29, 2019 at 4:27 AM Andres Freund <andres(at)anarazel(dot)de> wrote:
> On March 28, 2019 11:24:46 AM EDT, Peter Geoghegan <pg(at)bowt(dot)ie> wrote:
> >On Thu, Mar 28, 2019 at 5:28 AM Andres Freund <andres(at)anarazel(dot)de>
> >wrote:
> >> Hm, good catch. I don't like this fix very much (even if it were
> >> commented), but I don't have a great idea right now.
> >
> >That was just a POC, to verify the problem. Not a proposal.
>
> I'm mildly inclined to push a commented version of this. And add a open items entry. Alternatively I'm thinking of just but taking the tablespace setting into account.
I didn't understand that last sentence.
Here's an attempt to write a suitable comment for the quick fix. And
I suppose effective_io_concurrency is a reasonable default.
It's pretty hard to think of a good way to get your hands on the real
value safely from here. I wondered if there was a way to narrow this
to just GLOBALTABLESPACE_OID since that's where pg_tablespace lives,
but that doesn't work, we access other catalog too in that path.
Hmm, it seems a bit odd that 0 is supposed to mean "disable issuance
of asynchronous I/O requests" according to config.sgml, but here 0
will prefetch 10 buffers.
--
Thomas Munro
https://enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
0001-Fix-deadlock-in-heap_compute_xid_horizon_for_tuples.patch | text/x-patch | 1.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2019-03-30 12:16:21 | Re: pgsql: Improve autovacuum logging for aggressive and anti-wraparound ru |
Previous Message | Peter Eisentraut | 2019-03-30 07:25:44 | pgsql: Generated columns |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2019-03-30 10:56:27 | Re: REINDEX CONCURRENTLY 2.0 |
Previous Message | Fred .Flintstone | 2019-03-30 10:16:27 | Re: PostgreSQL pollutes the file system |