Log In - Home Page

CommitFest 2009-09 (Closed)

New Patch - Activity Log - CommitFest Topics

The most recent three comments for each patch will be displayed below. To view all the comments for a particular patch, or to add a comment or make other changes, click on the patch name.

Filtering on status "Returned with Feedback".

Status Summary. Committed: 20, Returned with Feedback: 14, Rejected: 6. Total: 40.

Returned with Feedback

Patch Name Status Author Reviewers Date Closed
Sequence ownership visibility functions
Patch by jwilliams on 2009-09-03: Initial version.
Comment by rhaas on 2009-09-14: Author agrees this should be marked returned with feedback.
Returned with Feedback Josh Williams Nobody 2009-09-14
FDW-based dblink
Patch by itagaki on 2009-08-19: Initial version.
Comment by petere on 2009-09-15: various concerns about interface and approach have been brought up
Returned with Feedback Itagaki Takahiro Peter Eisentraut 2009-09-15
opportunistic freezing during vacuum
Comment by selenamarie on 2009-09-14: Greg Stark reviewed, suggested change to remove a duplicated page scan. Jeff to submit an updated patch.
Comment by jdavis on 2009-09-15: I'm not actively working on this patch due to the difficulty in performance testing.
Comment by rhaas on 2009-09-17: Author agrees that this should be moved to Returned with Feedback.
Returned with Feedback Jeff Davis Jeff Janes 2009-09-17
Join optimization for inheritance tables
Patch by eulerto on 2009-08-10: Initial version.
Review by tgl on 2009-09-18: I don't feel that this is the direction we want to be taking with improving our partitioning logic.
Returned with Feedback Herodotos Herodotou, Nedyalko Borisov Tom Lane 2009-09-18
Separate Heap Fetch from Index Scan
Review by rhaas on 2009-09-14: Initial comments, mostly on joinpath.c changes.
Review by rhaas on 2009-09-20: Some ideas on planning where to put heap fetches.
Review by rhaas on 2009-09-20: Brief review of rest of patch.
Returned with Feedback Heikki Linnakangas Robert Haas 2009-09-20
Generalized Index Constraints
Comment by rhaas on 2009-09-27: Ping patch author: Need new version RSN.
Patch by jdavis on 2009-09-27: Updated patch, many changes, still some work to do.
Comment by rhaas on 2009-09-27: Still has a significant TODO list; should be postponed to next CF.
Returned with Feedback Jeff Davis Brendan Jurd 2009-09-27
Linux LSB init script
Comment by rhaas on 2009-09-17: Kevin will be AFK for a week or so
Comment by rhaas on 2009-09-20: What next?
Comment by rhaas on 2009-09-27: No progress, marking Returned with Feedback.
Returned with Feedback Kevin Grittner Peter Eisentraut 2009-09-27
new function for connect in libpq
Comment by jcasanov on 2009-09-14: Andrew Chernow remembers there is still no decision in the prototype for the function
Review by tgl on 2009-09-23: Some complaints about the implementation.
Comment by rhaas on 2009-09-27: sounds like this needs a major rewrite, so next CF
Returned with Feedback Jaime Casanova Nobody 2009-09-27
Review by jjanes on 2009-09-19: I do not think this is the right approach.
Comment by rhaas on 2009-09-27: Summary of issues, request for updated patch.
Comment by rhaas on 2009-10-03: Time to move this one to Returned with Feedback.
Returned with Feedback Zoltan Boszormenyi Jeff Janes 2009-10-03
Lock wait statistics
Review by jcasanov on 2009-09-28: we still have to get an agreement about what info we need to show
Review by rhaas on 2009-09-30: Jeff Janes thinks this has some incorrect logic.
Comment by rhaas on 2009-10-05: Moving to "Returned with Feedback".
Returned with Feedback Mark Kirkwood Jaime Casanova 2009-10-05
Enhancements to COPY (error logging and autopartitioning)
Review by selenamarie on 2009-10-04: Reviewed error logging portion of the patch
Patch by selenamarie on 2009-10-05: Updated patch
Review by rhaas on 2009-10-07: error_logging code should be split into multiple features, and other suggestions.
Returned with Feedback Emmanuel Cecchet Selena Deckelmann, Jeff Davis, Robert Haas 2009-10-07
Pgbench Shell command
Patch by michael-kun on 2009-09-18: patch version 4: error handling stuff added
Patch by michael-kun on 2009-09-24: setshell command feature added, which permits to load a parameter from a shell script output. Usefull for statistical models
Review by gsmith on 2009-10-08: Still needs a number of fixes, details in the message.
Returned with Feedback Michael Paquier Dan Colish, Gabrielle Roth, Greg Smith 2009-10-08
Buffer usage in EXPLAIN and pg_stat_statements
Patch by itagaki on 2009-10-04: Fix a bug and terminologies.
Review by tgl on 2009-10-13: Some questions ...
Comment by tgl on 2009-10-13: Labeling this as a contrib-module patch is 100% misleading, because all but a tiny fraction of it is touching the core. Moving to "Performance" for lack of a better topic.
Returned with Feedback Itagaki Takahiro Euler Taveira de Oliveira, Robert Haas 2009-10-14
Largeobject access controls
Patch by kaigai on 2009-10-07: Revised patch, according to Tom's comment
Patch by kaigai on 2009-10-12: Revised patch, due to the conflicts with CVS HEAD
Review by tgl on 2009-10-13: Still needs work :-(
Returned with Feedback KaiGai Kohei Jaime Casanova 2009-10-14