On Wed, Mar 25, 2015 at 10:53 AM, Andres Freund <andres@2ndquadrant.com> wrote: > > Hi, > > On 2015-03-25 11:38:30 +0900, Michael Paquier wrote: > > On Tue, Mar 24, 2015 at 8:46 PM, Thom Brown wrote: > > > The index is unlogged until reindexing... > > > > > > [...] > > > Which is think also raises the question, why are unlogged indexes made > > > persistent by a reindex? > > > > That's a bug of HEAD, ~9.4 keeping the index as unlogged even after > > REINDEX INDEX. What happens is that ReindexIndex relies on > > relpersistence provided by makeRangeVar at parse time, which is just > > incorrect as it uses RELPERSISTENCE_PERMANENT all the time. The patch > > attached fixes that... > > What the hell? That's somewhat nasty. Nice that it got caught before 9.5 > was released. >
Unfortunately this is very nasty. Sorry!
> Did you check whether a similar bug was made in other places of > 85b506bb? Could you additionally add a regression test to this end? > Seems like something worth testing. >