Re: pg_upgrade - link mode and transaction-wraparound data loss
Bruce Momjian <bruce@momjian.us>
From: Bruce Momjian <bruce@momjian.us>
To: Bruce Momjian <bruce@momjian.us>
Cc: Jesper Krogh <jesper@krogh.cc>, pgsql-hackers@postgresql.org
Date: 2010-05-19T18:29:30Z
Lists: pgsql-hackers
Attachments
- /pgpatches/pg_upgrade (text/x-diff) patch
Bruce Momjian wrote: > > This is the "production system". I have absolutely no indications that > > anything should be wrong in there. It has run rock-solid since it got > > migrated (dump/restore) to 8.4 for about 7 months now. So I am a bit > > scared about you telling that it seems wrong. (but that cannot be > > attributed to pg_upgrade) > > I am on chat with Alvaro now and it seems we do somehow connect to > template0 for transaction id wraparound. I think Alvaro will post > shortly on this. > > > > OK, thanks. This does seem odd. Frankly, having template0's > > > datfrozenxid be wrong would not cause any kind of instability because > > > template0 is used only by pg_dump, so I am wondering if something else > > > is seriously wrong. > > > > > I also think that something was seriously wrong with the pg_upgrade'd > > version. I'll try to reproduce and be a bit more carefull in tracking > > the steps > > this time. > > Thanks, but I think the entire problem might be this template0 xid issue > that Alvaro and I are researching. I can now see how invalid template0 > xids could cause the instability you saw in the new database. Odd no > one has seen this bug before. OK, after talking to Alvaro and Heikki, the problem is that while you cannot connect to template0, it is accessed by autovacuum for vacuum freeze, even if autovacuum is turned off. I think the reason you are seeing this bug is that your xid counter is near 2 billion (50% to wraparound) and the original template0 xids are the maximum distance from your counter. I am attaching the newest patch which fixes this issue. I did modify this code yesterday with another patch, and I am unclear exactly if you need that patch as well. CVS now has all these changes. If you could test with this and the earlier patch, I think it will now work fine. Thanks for the valuable testing, and quick feedback. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com