FAQ

PostgreSQL > pgsql-rrreviewers

Search Discussions

109 discussions - 468 posts

  • Hi, Our IT Company systems architecture is based on IBM Websphere Application Server, we would like to migrate our databases to Postgres, the main problem which stops us from doing that is Postgres ...
    John NashJohn Nash
    Oct 15, 2012 at 8:22 am
    Oct 15, 2012 at 8:22 am
  • There hasn't been any discussion of who will manage this CF that I've heard. Since nature abhors a vacuum, I took some preliminary steps to get it started, and will take the role if nobody objects ...
    Kevin GrittnerKevin Grittner
    Jun 16, 2012 at 6:16 pm
    Jun 16, 2012 at 6:16 pm
  • Today starts the last CommitFest for PostgreSQL 9.2. We currently have 83 patches that are looking for a reviewer at https://commitfest.postgresql.org/action/commitfest_view?id=13 ; anyone who has ...
    Greg SmithGreg Smith
    Jan 16, 2012 at 10:08 am
    Jan 16, 2012 at 10:08 am
  • The November CommitFest is now closed for new entries. We have 30 patches in the queue that are still looking for a reviewer at this point, out of a total of 53. If you'd like to review a patch but ...
    Greg SmithGreg Smith
    Nov 16, 2011 at 3:25 pm
    Nov 16, 2011 at 3:25 pm
  • I put myself down as a reviewer for the FlexLock patch. This seems big enough and important enough that I don't want to discourage anyone else from jumping in to do a review, also. -Kevin
    Kevin GrittnerKevin Grittner
    Nov 15, 2011 at 10:05 pm
    Nov 15, 2011 at 10:05 pm
  • Please give me the simpler patches -- I've been using pgsql for years, but only read some pgsql code a couple of months ago ;) I'm not strong in C either, but I'll nut it out.
    Royce AusburnRoyce Ausburn
    Nov 14, 2011 at 9:27 pm
    Nov 15, 2011 at 2:16 am
  • This is primarily to reassure those concerned about the CF process that I am indeed following up on things off-list. It is also to reassure those whom I've nagged that you're not alone. But with the ...
    Kevin GrittnerKevin Grittner
    Sep 26, 2011 at 12:04 am
    Sep 26, 2011 at 12:04 am
  • Hi, is the commit-fest already launched, isn't it? I try to review Heikki's "Inserting heap tuples in bulk in COPY" patch. Could you wait for my comment for a few days. Thanks, -- KaiGai Kohei ...
    Kohei KaiGaiKohei KaiGai
    Sep 16, 2011 at 12:57 pm
    Sep 18, 2011 at 3:56 pm
  • All, We're almost at the end of the CommitFest, and three patches have not yet been reviewed: lazy vxid locks https://commitfest.postgresql.org/action/patch_view?id=585 sepgsql - userspace access ...
    Josh BerkusJosh Berkus
    Jul 14, 2011 at 12:42 am
    Jul 15, 2011 at 7:27 pm
  • Hackers, Commitfest 2 continues to progress ... slowly. At this point, we have no hope of wrapping it up early; my best hope is to at least finish on time. Statistics: * 1/2 of patches are still ...
    Josh BerkusJosh Berkus
    Jul 7, 2011 at 4:02 am
    Jul 7, 2011 at 6:19 pm
  • Reviewers, You are receiving this e-mail because you volunteered to review one or more patches in the current commitfest, and have not posted a review or update of those after 5 days or more. Given ...
    Josh BerkusJosh Berkus
    Jun 27, 2011 at 8:22 pm
    Jun 30, 2011 at 8:21 am
  • All, For easy visibility, I've moved all "WIP" patches to their own section in the current commitfest, at the bottom of the list of pending patches. Hopefully this way there will be less confusion ...
    Josh BerkusJosh Berkus
    Jun 17, 2011 at 9:30 pm
    Jun 17, 2011 at 9:30 pm
  • Folks, I realize that a bunch of our regulars are busy with wrapping up 9.1. However, I'm sure that there were more than 3 round robin reviewers for the 9.1 CFs. Where are you hiding? 9.2 CF1 is ...
    Joshua BerkusJoshua Berkus
    Jun 16, 2011 at 4:44 pm
    Jun 16, 2011 at 5:31 pm
  • RRReviewers, Our first CommitFest for 9.1 starts in a couple weeks. Since y'all have had a few months off, I need to know who is available for assignment of random patches. Of course, if you want to ...
    Josh BerkusJosh Berkus
    Jun 3, 2011 at 11:43 pm
    Jun 26, 2011 at 7:53 pm
  • I think this patch missed the deadline for the current CommitFest. It's true that it was posted to the list in time, but it's just madness to think we can do review in a meaningful way and get done ...
    Robert HaasRobert Haas
    Feb 8, 2011 at 3:46 am
    Feb 8, 2011 at 10:42 pm
  • With ten days left in the current CommitFest, being the last CommitFest for 9.1 development, there are presently 40 patches that are marked either Needs Review or Waiting on Author. The 11 patches ...
    Robert HaasRobert Haas
    Feb 4, 2011 at 3:29 pm
    Feb 4, 2011 at 5:03 pm
  • PgUS is preparing to fund a grant for PgUS members to learn and participate in the patch review process. We looking for experienced reviewers that can assist a candidate through to process of testing ...
    Richard BroersmaRichard Broersma
    Jan 24, 2011 at 8:17 pm
    Jan 27, 2011 at 12:28 pm
  • We have 46 patches in this CommitFest so far and I know that there are quite a few patches that have been posted but not added to the CommitFest application yet (please fix this, if you are a patch ...
    Robert HaasRobert Haas
    Jan 12, 2011 at 2:17 am
    Feb 16, 2011 at 3:37 pm
  • It's the time of year again where leaves are falling, temperatures dropping, and patches cry out looking for one last review before the start of the holiday season. The patches submitted so far are ...
    Greg SmithGreg Smith
    Nov 13, 2010 at 12:08 am
    Nov 18, 2010 at 1:09 am
  • Guys, I'm in for the glory of reviewing patches (special thanks to David Fetter on paying me with a brand new BMW for reviewing) Any other spanish speaking patch reviewer around? gb.-
    ๏̯͡๏ Guido Barosio๏̯͡๏ Guido Barosio
    Oct 1, 2010 at 5:48 pm
    Oct 2, 2010 at 12:26 am
  • David, Could you assign me a next patch to be reviewed? My hands become free now, since LinuxCon Japan was closed yeaterday. (I had two sessions in this conference...) Thanks, -- KaiGai Kohei ...
    KaiGai KoheiKaiGai Kohei
    Sep 30, 2010 at 6:25 am
    Sep 30, 2010 at 1:21 pm
  • This is a basic review of Fujii Masao's synchronous replication patch from http://archives.postgresql.org/message-id/AANLkTik2c3kV7HgJnM4MjkCWVG-QvDJXD3iR9TqsCnpP@mail.gmail.com Review Description ...
    Thom BrownThom Brown
    Sep 29, 2010 at 10:30 pm
    Sep 30, 2010 at 12:39 am
  • It's been about a week now. Should we expect a new patch soon? Regards, Marko Tiikkaja
    Marko TiikkajaMarko Tiikkaja
    Sep 29, 2010 at 7:14 pm
    Oct 5, 2010 at 1:49 am
  • Folks, We're almost half way through the commitfest, and so I'll start with: The Good: - Most patches still in play have a reviewer. - It's possible for one person to post 5 reviews in a day. Robert ...
    David FetterDavid Fetter
    Sep 28, 2010 at 9:03 pm
    Sep 29, 2010 at 2:16 am
  • [review elided] Pavel, Will you have time to get this cleaned up this week per Hitoshi's review? If not, I'll mark it "returned with feedback," and move it to the next Commitfest. Cheers, David. -- ...
    David FetterDavid Fetter
    Sep 20, 2010 at 5:11 am
    Sep 20, 2010 at 6:23 am
  • Stephen, I've assigned this to you :) https://commitfest.postgresql.org/action/patch_view?id=366 Cheers, David. -- David Fetter <david@fetter.org http://fetter.org/ Phone: +1 415 235 3778 AIM: ...
    David FetterDavid Fetter
    Sep 17, 2010 at 4:53 pm
    Sep 17, 2010 at 4:53 pm
  • Hi reviewers, I'm intending to post initial reviews of the following over the next few days: * WIP: extensible enums * Basic JSON support * string function II CF app has been updated. Cheers, BJ
    Brendan JurdBrendan Jurd
    Sep 16, 2010 at 6:52 am
    Sep 16, 2010 at 6:52 am
  • Folks, If it weren't for the last minute, nothing would ever get done. In the hallowed tradition of Commitfests, and the Feature Freezes that preceded them, the past week has been a flurry of ...
    David FetterDavid Fetter
    Sep 16, 2010 at 6:07 am
    Sep 16, 2010 at 7:49 pm
  • So did the materialized views patch not get submitted? -- Thom Brown Twitter: @darkixion IRC (freenode): dark_ixion Registered Linux user: #516935
    Thom BrownThom Brown
    Sep 13, 2010 at 11:45 am
    Sep 13, 2010 at 12:00 pm
  • Folks, The Commitfest for September, 2010, will start in just a couple of days, and you can help make it successful! The first way is to volunteer for patch-chasing, as Selena, Stephen and Brendan ...
    David FetterDavid Fetter
    Sep 13, 2010 at 4:55 am
    Sep 13, 2010 at 4:55 am
  • The second commit fest shall be launched on the next Wednesday. Since I finished to submit all the patches that I planned in this commit fest, I'll come into the task reviewing patches. At first, I'd ...
    KaiGai KoheiKaiGai Kohei
    Sep 9, 2010 at 1:46 am
    Sep 9, 2010 at 1:51 am
  • I think this has been obsoleted by the xmlexists patch.
    Peter EisentrautPeter Eisentraut
    Aug 6, 2010 at 4:38 am
    Aug 8, 2010 at 7:18 pm
  • "Kevin Grittner" 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 ...
    Kevin GrittnerKevin Grittner
    Aug 5, 2010 at 6:43 pm
    Aug 18, 2010 at 9:46 pm
  • Hi, If no one objects i will take the following patch: Distinguish between unique indexes and unique constraints But i will not be able to review it until week end, so if one wants to take a look ...
    Jaime CasanovaJaime Casanova
    Jul 29, 2010 at 7:02 am
    Jul 29, 2010 at 7:02 am
  • The "Add xml_is_well_formed Function" by Mike Fowler can't be reviewed by the person who initially signed on; if you're looking for a patch to review, this one is in the mix again. -Kevin
    Kevin GrittnerKevin Grittner
    Jul 28, 2010 at 1:58 pm
    Jul 28, 2010 at 1:58 pm
  • == Submission review == * Is the patch in context diff format? Yes. * Does it apply cleanly to the current CVS HEAD? Yes. patch -p1 < ../xpath_exists-3.patch patching file doc/src/sgml/func.sgml Hunk ...
    David FetterDavid Fetter
    Jul 27, 2010 at 11:33 pm
    Jul 27, 2010 at 11:56 pm
  • If you've been considering what to review, and haven't felt that any of the unassigned patches were a good fit for you, please note that the "include Backend ID in relpath for temp rels" patch by ...
    Kevin GrittnerKevin Grittner
    Jul 21, 2010 at 2:09 pm
    Jul 21, 2010 at 2:09 pm
  • I've just spent hours poring over CF entries and old emails to see where we are at with all the patches. I've sent out a dozen off-list emails to try to keep the CF moving. To stretch the limits of ...
    Kevin GrittnerKevin Grittner
    Jul 20, 2010 at 10:29 pm
    Jul 20, 2010 at 10:29 pm
  • I'd like to mark this as Ready for Committer :) Review below. Cheers, David. == Submission review == * Is the patch in context diff format? Yes. * Does it apply cleanly to the current CVS HEAD? Yes, ...
    David FetterDavid Fetter
    Jul 19, 2010 at 8:18 pm
    Jul 19, 2010 at 8:18 pm
  • Folks, I'd like to take a whack at this one :) Cheers, David. -- David Fetter <david@fetter.org http://fetter.org/ Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter Skype: davidfetter XMPP: ...
    David FetterDavid Fetter
    Jul 19, 2010 at 7:03 pm
    Jul 19, 2010 at 7:03 pm
  • Hi, I'm done with my review and I'd like to pick up the next patch. I could try my luck with gincostestimate, I already had some exposure to cost estimation functions. I'll set myself as the reviewer ...
    Jan UrbańskiJan Urbański
    Jul 19, 2010 at 6:51 pm
    Jul 19, 2010 at 6:51 pm
  • /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 ...
    Kevin GrittnerKevin Grittner
    Jul 15, 2010 at 2:44 pm
    Jul 29, 2010 at 3:46 pm
  • Folks, The PostgreSQL 9.1 Development Plan: http://wiki.postgresql.org/wiki/PostgreSQL_9.1_Development_Plan calls for a CommitFest to run from the 15th of July until the 15th of August. I've offered ...
    Kevin GrittnerKevin Grittner
    Jul 12, 2010 at 1:25 pm
    Jul 12, 2010 at 4:44 pm
  • Hello are somebody working on patch https://commitfest.postgresql.org/action/patch_view?id=326 ? If not I can do review this weekend. Regards Pavel Stehule
    Pavel StehulePavel Stehule
    Jul 9, 2010 at 9:39 am
    Jul 9, 2010 at 12:40 pm
  • Who is going to manage the actual CF? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company
    Robert HaasRobert Haas
    Jul 7, 2010 at 11:09 pm
    Jul 8, 2010 at 5:03 pm
  • Perhaps, my hands will be free until Peter updates the unix domain socket authentication patch. http://archives.postgresql.org/message-id/4C2005AF.7090300@ak.jp.nec.com So, I'd like to pick up a next ...
    KaiGai KoheiKaiGai Kohei
    Jun 24, 2010 at 12:51 am
    Jun 24, 2010 at 2:59 am
  • Don't forget to change the status when appropriate, which usually is when a review is posted or a new patch is submitted after a review. To do this, click on the "Edit Patch" link. Thanks, -Kevin
    Kevin GrittnerKevin Grittner
    Jun 16, 2010 at 6:50 pm
    Jun 16, 2010 at 6:50 pm
  • Folks, The PostgreSQL 9.1 Development Plan: http://wiki.postgresql.org/wiki/PostgreSQL_9.1_Development_Plan calls for a ReviewFest to run from the 15th of June (tomorrow) until the start of the first ...
    Kevin GrittnerKevin Grittner
    Jun 14, 2010 at 4:25 pm
    Jun 15, 2010 at 11:59 pm
  • Hi! During the Developers meeting at PgCon, the group at the meeting agreed to maintain last year's development schedule. This means that the first commitfest of the year will start on July 15 ...
    Selena DeckelmannSelena Deckelmann
    May 20, 2010 at 9:21 pm
    May 24, 2010 at 7:09 pm
  • Hi, I just edited the following commitfest page to orphan the patch I had been assigned to: https://commitfest.postgresql.org/action/patch_view?id=242 The current sleeping pattern of my son evolves ...
    Dimitri FontaineDimitri Fontaine
    Jan 30, 2010 at 9:53 pm
    Feb 1, 2010 at 12:16 am
Latest
Group Navigation
period‹ prev | Latest | first ›
Group Overview
grouppgsql-rrreviewers @
categoriespostgresql
discussions109
posts468
users71
websitepostgresql.org...
irc#postgresql

Top users

Robert Haas: 96 posts Josh Berkus: 46 posts Greg Smith: 31 posts Jeff Davis: 26 posts Jaime Casanova: 25 posts Kevin Grittner: 24 posts David Fetter: 20 posts Dave Page: 19 posts Richard Broersma: 12 posts Ibrar Ahmed: 8 posts Alex Hunsaker: 7 posts Dimitri Fontaine: 7 posts Kenneth Marshall: 6 posts David E. Wheeler: 6 posts KaiGai Kohei: 6 posts Tom Lane: 5 posts Pavel Stehule: 5 posts Jaime Casanova: 5 posts Stephen Frost: 5 posts Brendan Jurd: 5 posts
show more
Archives