Thread: CommitFest 2010-07 now in progress
/me bangs gavel I hereby declare the 2010-07 CommitFest closed to further patch submissions, as it is now officially "In Progress". We have one month to provide initial review of the patches which have accumulated since the start of the last CommitFest, six months ago, and hopefully get a reasonable number of them committed. All reviewers currently assigned to a patch in "Needs Review" status should post a review within the next four days. All authors with patches in "Waiting on Author" status should post a response within four days. If there's a reason that can't happen, please let me know off-list. Some numbers: 68 patches were submitted 3 patches were withdrawn (deleted) by their authors -- 65 total patches currently in the application -- 3 committed to 9.0 -- 62 9.1 patches -- 1 rejected 3 returned with feedback 1 committed for 9.1 -- 57 pending 10 ready for committer -- 47 will still need reviewer attention 6 waiting on author to respond to review -- 41 need review before further action 23 patches "Needs Review" patches don't have a reviewer assigned -- 18 patches have reviews due within four days We could still use additional reviewers. It's not too late to sign up! To get an idea of what's involved, please read these pages: http://wiki.postgresql.org/wiki/Reviewing_a_Patch http://wiki.postgresql.org/wiki/RRReviewers On the lighter side: http://wiki.postgresql.org/images/5/58/11_eggyknap-patch-review.pdf Please send me an email (without copying the list) if you are available to review; feel free to include any information that might be helpful in assigning you an appropriate patch. To see what patches still need a reviewer, you could scroll through the web application looking at the "Reviewers" column: http://commitfest.postgresql.org/action/commitfest_view/inprogress -Kevin
New numbers on where we are with this CommitFest: 71 patches were submitted 3 patches were withdrawn (deleted) by their authors -- 68 total patches currently in the application -- 3 committed to 9.0 -- 65 9.1 patches -- 1 rejected 5 returned with feedback 11 committed for 9.1 -- 17 9.1 patches disposed -- 48 pending 8 ready for committer -- 40 will still need reviewer attention 9 waiting on author to respond to review -- 31 need review before further action 13 "Needs Review" patches don't have a reviewer assigned -- 18 patches have reviews due within four days or less I had particular concerns about the synchronous replication patches, because I've seen competing patches like that lead to significant wheel-spinning and duplication of effort. To try to minimize that, I asked Yeb to do a preliminary review of both. He has not completed reporting on that, but expects to finish within a day or two. My hope is that all parties who want to move this forward will join efforts and bring their ideas to bear on a single patch. I see that nobody has signed up for the GSoC patch regarding materialized views. Do we have an official "mentor" for this effort? Would it be a good or bad idea for that person to do the review? Although we've had some discussion around Markus Wanner's WIP refactoring patches and the prerequisite miscellaneous patches, there's nobody down as a Reviewer for any of them. I understand that the six WIP patches are there for feedback, not with expectation of a commit in this CF, but I'm less clear about the two prerequisite patches. The "WIP patch for serializable transactions with predicate locking" is big, but there's no expectation of a commit this CF, and Joe assures me he's working on it every day; so nobody should be concerned about the lack of a review post on that so far. I am hopeful that in the next week we can clear a lot of the pending patches which have already had some review, and get someone on every unclaimed patch. -Kevin "Kevin Grittner" <Kevin.Grittner@wicourts.gov> wrote: > 68 patches were submitted > 3 patches were withdrawn (deleted) by their authors > -- > 65 total patches currently in the application > -- > 3 committed to 9.0 > -- > 62 9.1 patches > -- > 1 rejected > 3 returned with feedback > 1 committed for 9.1 > -- > 57 pending > 10 ready for committer > -- > 47 will still need reviewer attention > 6 waiting on author to respond to review > -- > 41 need review before further action > 23 patches "Needs Review" patches don't have a reviewer assigned > -- > 18 patches have reviews due within four days
On Thu, Jul 22, 2010 at 2:09 PM, Kevin Grittner <Kevin.Grittner@wicourts.gov> wrote: > 48 pending > 8 ready for committer Note that all of the patches except one which are marked as "Ready for Committer" were either submitted by a committer, or the reviewer is a committer. Of those, 3 are mine. Two of those are patches I'm postponing committing at the request of Tom Lane to avoid making the 9.1 and 9.0 trees drift too much before 9.0 is out. However, given the rapidly decreasing frequency of commits to the 9.0 branch, I'm not sure how much longer it makes sense to hold off: I'm currently thinking I'll commit those two after beta4 wraps. The last of those is the 5-key syscaches patch, which only makes sense if knngist needs it, so it may get bumped to the next CF, as knngist was not submitted in time for this CF. The other 4 patches were either submitted or reviewed by Simon Riggs or Itagaki Takahiro, and I am presuming they will commit them themselves unless I hear otherwise (in which case I'm happy to pick them up). That leaves just one patch that's actually been reviewed and is ready to be picked up by a committer, so we actually have a bit of a pipelines stall here. > 18 patches have reviews due within four days or less This is a very big number... I hope some of these reviews start to come in soon. I think this is where our bottleneck is at present. > Although we've had some discussion around Markus Wanner's WIP > refactoring patches and the prerequisite miscellaneous patches, > there's nobody down as a Reviewer for any of them. I understand > that the six WIP patches are there for feedback, not with > expectation of a commit in this CF, but I'm less clear about the two > prerequisite patches. It seems to me that the discussion is Alvaro and I are having with Markus is tilted toward having Markus rewrite the imessages interface to use an SLRU, in which case neither of them will go in this CF. I'm hopeful that Heikki or Tom will comment on this also when they get back from their vacations. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company
Robert Haas <robertmhaas@gmail.com> wrote: >> 18 patches have reviews due within four days or less > > This is a very big number... I hope some of these reviews start to > come in soon. I think this is where our bottleneck is at present. Based on off-list emails, I expect most of these to clear by this weekend. Part of this was caused by people "reserving" several patches up front, and posting reviews on some but just now getting to others; part has been caused by people traveling, and not being at "home base" to work on things; part has been due to high priority non-PostgreSQL issues taking people away from reviewing for a few days; and the predicate locking/serializable patch is just too big to review in a few days and I didn't bother taking it out of the count. You'd probably feel better about things if you had read all the off-list emails. Not that we couldn't use another reviewer or two. I'm still welcoming volunteers! -Kevin
Hi, On 07/22/2010 08:51 PM, Robert Haas wrote: > It seems to me that the discussion is Alvaro and I are having with > Markus is tilted toward having Markus rewrite the imessages interface > to use an SLRU, in which case neither of them will go in this CF. I'm > hopeful that Heikki or Tom will comment on this also when they get > back from their vacations. Just for the record: I don't currently think a rewrite to use SLRU makes any sense for imessages. But to answer Kevin's question: I don't expect to have the prerequisite patches committed this CF, as I don't think it currently makes any sense for Postgres to apply them. Nor did I feel there's general consensus that having we want to have a dynamic memory allocator (for shared memory). It would be nice to be able to keep track of these kind of patches, which are available to Postgres and get maintained, but aren't currently needed or wanted. But do we want to use the CF application for that? How do you prefer to proceed with these patches? It's also worth noting that Simon requested more and better documentation. But I simply can't promise to write anything soon. Regards Markus Wanner
New numbers on where we are with this CommitFest, as we approach the half-way point: 72 patches were submitted 3 patches were withdrawn (deleted) by their authors 8 patches were moved to CommitFest 2010-09 -- 61 patches in CommitFest 2010-07 -- 3 committed to 9.0 -- 58 patches for 9.1 -- 1 rejected 13 returned with feedback 12 committed for 9.1 -- 26 disposed -- 32 pending 10 ready for committer -- 22 will still need reviewer attention 7 waiting on author to respond to review -- 15 need review before further action 2 "Needs Review" patches don't have a reviewer assigned -- 13 patches need review and have a reviewer assigned Of the eight patches moved to the next CF, all were moved by or at the request of their authors. One was because the author didn't feel the patch was ready for review and didn't have time to take care of that in this CF. Six were WiP patches which need documentation (perhaps a Wiki page) before others can effectively review them. One is ready for committer, but isn't needed until we are ready to commit the KNN-GiST, which was submitted for the next CF. 13 of the 22 patches which will still need reviewer attention have had at least one review. Many of the others have had discussion and comment entries, but not yet a formal review. The "WIP patch for serializable transactions with predicate locking" by Dan Ports and myself has had some off-list questions from Joe Conway. The questions are noted as opportunities for further code comments. He pointed out one bug which has been fixed. And the questions have caused me to notice a couple areas which need work to reduce the false positive rate. The last two patches which are without an assigned reviewer appear to be in that state because there aren't many people who feel competent to review these areas. The "ECPG FETCH readahead" patch by Zoltán Böszörményi and the "WiP: Per-column collation" patch by Peter Eisentraut both need *someone* to step up. Volunteers or suggestions welcome. Perhaps the biggest CF news of the last week is that we are no longer faced with a fork in the efforts to implement synchronous replication for 9.1 -- Zoltán Böszörményi has heroically offered to withdraw his patch and work with Fujii Masao on enhancing the subsequent "Another synchronous replication" patch. With everyone working from the same base to push this effort forward, I'm hopeful that we can overcome the challenges this technology presents. I think it will be very good for the project if we can get a fairly polished and "close to final" version committed before the last CommitFest, so that it has a full alpha test cycle to settle in. Note that this means that such a patch must be submitted within *three and a half months*! Yes, we are that far in to the 9.1 development cycle. Some of the other patches may have funny dates, but I believe from off-list emails that things are generally moving OK. -Kevin "Kevin Grittner" <Kevin.Grittner@wicourts.gov> wrote: > 71 patches were submitted > 3 patches were withdrawn (deleted) by their authors > -- > 68 total patches currently in the application > -- > 3 committed to 9.0 > -- > 65 9.1 patches > -- > 1 rejected > 5 returned with feedback > 11 committed for 9.1 > -- > 17 9.1 patches disposed > -- > 48 pending > 8 ready for committer > -- > 40 will still need reviewer attention > 9 waiting on author to respond to review > -- > 31 need review before further action > 13 "Needs Review" patches don't have a reviewer assigned > -- > 18 patches have reviews due within four days or less
"Kevin Grittner" <Kevin.Grittner@wicourts.gov> wrote: New numbers on where we are with this CommitFest, at the end of the third week: 72 patches were submitted 3 patches were withdrawn (deleted) by their authors 12 patches were moved to CommitFest 2010-09 -- 57 patches in CommitFest 2010-07 -- 3 committed to 9.0 -- 54 patches for 9.1 -- 1 rejected 17 returned with feedback 21 committed for 9.1 -- 39 disposed -- 15 pending 9 ready for committer -- 6 will still need reviewer attention 1 waiting on author to respond to review -- 5 patches need review now and have a reviewer assigned Of the four patches moved to the next CF, one was because we couldn't find a reviewer for ECPG code at this time, one was because both Florian and I would like to work up some additional tests for the "serializable lock consistency" patch before sending it to a committer, and two were because Itagaki changed jobs and didn't have time during this CF to finish reviews already well underway. With only ten days to go, in order to leave time for committers to do their thing, we need to be wrapping up the remaining patches. I think we look pretty good. Of the remaining six patches, two are Work in Progress, so are not expected to go to a committer; three involve a committer, so I figure they can decide when and if it's time to return or move them, which just leaves one which is down to tweaking docs. The "WIP patch for serializable transactions with predicate locking" patch has yet to have a review posted, although there have been off-list discussions. The reviewer had to put it aside for about a week due to job pressures, but is reported back on it. (The suspense is killing me.) Last week: > 72 patches were submitted > 3 patches were withdrawn (deleted) by their authors > 8 patches were moved to CommitFest 2010-09 > -- > 61 patches in CommitFest 2010-07 > -- > 3 committed to 9.0 > -- > 58 patches for 9.1 > -- > 1 rejected > 13 returned with feedback > 12 committed for 9.1 > -- > 26 disposed > -- > 32 pending > 10 ready for committer > -- > 22 will still need reviewer attention > 7 waiting on author to respond to review > -- > 15 need review before further action > 2 "Needs Review" patches don't have a reviewer assigned > -- > 13 patches need review and have a reviewer assigned
New numbers on where we are with this CommitFest, at the end of the fourth week: 72 patches were submitted 3 patches were withdrawn (deleted) by their authors 12 patches were moved to CommitFest 2010-09 -- 57 patches in CommitFest 2010-07 -- 3 committed to 9.0 -- 54 patches for 9.1 -- 1 rejected 18 returned with feedback 28 committed for 9.1 -- 47 disposed -- 7 pending 2 ready for committer -- 5 will still need reviewer attention 1 waiting on author to respond to review -- 4 patches need review now and have a reviewer assigned With about 56 hours left until the close of the CommitFest, we're down to two "Ready for Committer" and three other potentially committable patches. Do we have a plan (with a time line) for producing an 9.1alpha1 release after the CF closes? (Have we even set a policy on whether we do that when we're still in beta testing for the release which hit feature freeze six months ago?) No patches were moved to the next CF this week, seven were committed, and one was returned with feedback. Two of the four patches in "Needs Review" status are WiP patches, and for both a review is long overdue by CF guidelines. The other two in "Needs Review" status had new patches posted yesterday which respond to committer feedback. The last action for the one patch in "Waiting on Author" status was feedback from Tom five days ago. Last week: > 72 patches were submitted > 3 patches were withdrawn (deleted) by their authors > 12 patches were moved to CommitFest 2010-09 > -- > 57 patches in CommitFest 2010-07 > -- > 3 committed to 9.0 > -- > 54 patches for 9.1 > -- > 1 rejected > 17 returned with feedback > 21 committed for 9.1 > -- > 39 disposed > -- > 15 pending > 9 ready for committer > -- > 6 will still need reviewer attention > 1 waiting on author to respond to review > -- > 5 patches need review now and have a reviewer assigned
At the close of the 2010-07 CommitFest, the numbers were: 72 patches were submitted 3 patches were withdrawn (deleted) by their authors 14 patches were moved to CommitFest 2010-09 -- 55 patches in CommitFest 2010-07 -- 3 committed to 9.0 -- 52 patches for 9.1 -- 1 rejected 20 returned with feedback 31 committed for 9.1 When we hit the end of the allotted time, I moved the last two patches to the next CF, for want of a better idea for disposition. One is "Ready for Committer" with an author who is a committer. The other is my WiP patch for serializable transactions -- there's a lot to review and the reviewer had unexpected demands on his time during the CF; he said he'll continue work on that outside the CF. -Kevin At the end of week four: > 72 patches were submitted > 3 patches were withdrawn (deleted) by their authors > 12 patches were moved to CommitFest 2010-09 > -- > 57 patches in CommitFest 2010-07 > -- > 3 committed to 9.0 > -- > 54 patches for 9.1 > -- > 1 rejected > 18 returned with feedback > 28 committed for 9.1 > -- > 47 disposed > -- > 7 pending > 2 ready for committer > -- > 5 will still need reviewer attention > 1 waiting on author to respond to review > -- > 4 patches need review now and have a reviewer assigned
On 18 August 2010 22:45, Kevin Grittner <Kevin.Grittner@wicourts.gov> wrote: > At the close of the 2010-07 CommitFest, the numbers were: > > 72 patches were submitted > 3 patches were withdrawn (deleted) by their authors > 14 patches were moved to CommitFest 2010-09 > -- > 55 patches in CommitFest 2010-07 > -- > 3 committed to 9.0 > -- > 52 patches for 9.1 > -- > 1 rejected > 20 returned with feedback > 31 committed for 9.1 > > When we hit the end of the allotted time, I moved the last two > patches to the next CF, for want of a better idea for disposition. > One is "Ready for Committer" with an author who is a committer. The > other is my WiP patch for serializable transactions -- there's a lot > to review and the reviewer had unexpected demands on his time during > the CF; he said he'll continue work on that outside the CF. > > -Kevin > > > At the end of week four: > >> 72 patches were submitted >> 3 patches were withdrawn (deleted) by their authors >> 12 patches were moved to CommitFest 2010-09 >> -- >> 57 patches in CommitFest 2010-07 >> -- >> 3 committed to 9.0 >> -- >> 54 patches for 9.1 >> -- >> 1 rejected >> 18 returned with feedback >> 28 committed for 9.1 >> -- >> 47 disposed >> -- >> 7 pending >> 2 ready for committer >> -- >> 5 will still need reviewer attention >> 1 waiting on author to respond to review >> -- >> 4 patches need review now and have a reviewer assigned So did the materialized views patch not get submitted? -- Thom Brown Twitter: @darkixion IRC (freenode): dark_ixion Registered Linux user: #516935
On Mon, Sep 13, 2010 at 7:44 AM, Thom Brown <thom@linux.com> wrote: > So did the materialized views patch not get submitted? I think someone else will need to pick it up and do a bunch more work with it before it can be considered a serious candidate for commit. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company