Log In - Home Page

CommitFest 2013-01 (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 "Rejected".

Status Summary. Committed: 70, Returned with Feedback: 14, Rejected: 4. Total: 88.

Rejected Patches

Patch Name Status Author Reviewers Date Closed
Logging Collector Extensions
Patch by fdr on 2012-12-08: Initial version.
Patch by fdr on 2012-12-08: v2 - small error handling amendment
Comment by fdr on 2012-12-10: Discovered duplicate funtionality
Rejected Daniel Farina Nobody 2012-12-10
Remove PD_ALL_VISIBLE
Comment by ringerc on 2013-01-21: Looks like it might get bounced to next CF, but holding it a bit longer
Patch by jdavis on 2013-01-22: Fixed a bug, but it looks like the end of the line for this patch, anyway.
Comment by heikki on 2013-01-23: Marking rejected, as the consensus seems to be that we don't want this.
Rejected Jeff Davis Pavan Deolasee 2013-01-23
create extension default_full_version (from 2012-11)
Comment by ringerc on 2013-01-18: Emailed Dimitri to check patch state
Comment by ringerc on 2013-01-18: Dimitri notes that this patch depends on Extension Templates, so hold pending movement there. See https://commitfest.postgresql.org/action/patch_view?id=1032
Patch by dim on 2013-02-22: The new extension templates patch had to integrate default_major_version for pg_dump.
Rejected Dimitri Fontaine Ibrar Ahmed 2013-02-22
Add code commentary in ProcessInterrupts
Patch by ams on 2013-01-17: Minor wordsmithing
Comment by tgl on 2013-01-19: It looks like we need to fix the code, not just comment it better, so I'm not sure there's much point in committing this patch.
Comment by ringerc on 2013-03-04: Flagged rejected as it's been inactive for a few weeks and doesn't look promising.
Rejected Pavan Deolasee Gurjeet Singh, Stephen Frost, Abhijit Menon-Sen 2013-03-04