Re: [PATCHES] Infrastructure changes for recovery (v8)
От | Simon Riggs |
---|---|
Тема | Re: [PATCHES] Infrastructure changes for recovery (v8) |
Дата | |
Msg-id | 1229568649.4793.296.camel@ebony.2ndQuadrant обсуждение исходный текст |
Ответ на | Re: [PATCHES] Infrastructure changes for recovery (v8) (Alvaro Herrera <alvherre@commandprompt.com>) |
Ответы |
Re: [PATCHES] Infrastructure changes for recovery (v8)
|
Список | pgsql-hackers |
On Wed, 2008-12-17 at 23:32 -0300, Alvaro Herrera wrote: > Simon Riggs escribió: > > > Please let me know how I can make the reviewer's job easier. Diagrams, > > writeups, whatever. Thanks, > > A link perhaps? There is much confusion on this point for which I'm very sorry. I originally wrote "infra" patch to allow it to be committed separately in the Sept commitfest, to reduce size of the forthcoming hotstandby patch. That didn't happen (no moans there) so the eventual "hotstandby" patch includes all of what was the infra patch, plus the new code. So currently there is no separate "infra" patch. The two line items on the CommitFest page are really just one large project. I would be in favour of removing the "infra" lines from the CommitFest page. Of course we can consider "hotstandby" patch in parts, but deconstructing it into wholly separate patches doesn't make much sense now and would raise many questions about why certain code exists with no apparent function or why certain design choices made. If you were to review a part of this, I might ask that you look at the changes to XidInMVCCSnapshot(), GetSnapshotData() and AssignTransactionId(), which relate specifically to subtransaction handling. Comments explain the new approach. -- Simon Riggs www.2ndQuadrant.comPostgreSQL Training, Services and Support
В списке pgsql-hackers по дате отправления: