From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Andres Freund <andres(at)2ndquadrant(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Jim Nasby <jim(at)nasby(dot)net>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: REINDEX CONCURRENTLY 2.0 |
Date: | 2014-11-13 01:25:37 |
Message-ID: | 20141113012537.GA1791@alvin.alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andres Freund wrote:
> On 2014-11-12 18:23:38 -0500, Robert Haas wrote:
> > > The problem is that it's very hard to avoid the wrong index's
> > > relfilenode being used when swapping the relfilenodes between two
> > > indexes.
> >
> > How about storing both the old and new relfilenodes in the same pg_class entry?
>
> That's quite a cool idea
>
> [think a bit]
>
> But I think it won't work realistically. We have a *lot* of
> infrastructure that refers to indexes using it's primary key.
Hmm, can we make the relmapper do this job instead of having another
pg_class column? Essentially the same sketch Robert proposed, instead
we would initially set relfilenode=0 and have all onlookers use the
relmapper to obtain the correct relfilenode; switching to the new
relfilenode can be done atomically, and un-relmap the index once the
process is complete.
The difference from what Robert proposes is that the transient state is
known to cause failures for anyone not prepared to deal with it, so it
should be easy to spot what places need adjustment.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2014-11-13 01:26:49 | Re: REINDEX CONCURRENTLY 2.0 |
Previous Message | Andreas Karlsson | 2014-11-13 01:03:20 | Re: Using 128-bit integers for sum, avg and statistics aggregates |