Log In - Home Page

CommitFest 2014-12 (In Progress)

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.

Status Summary. Needs Review: 34, Waiting on Author: 6, Ready for Committer: 5, Committed: 16, Returned with Feedback: 12, Rejected: 5. Total: 78.

Pending Patches

Patch Name Status Author Reviewers Last Activity

Server Features

 
Foreign table inheritance
Comment by michael-kun on 2014-12-22: Patch for CHECK constraints has been committed. Not looked at 2nd patch yet.
Patch by efujita on 2014-12-22: Updated 2nd patch correspondingly to a fix for EXPLAIN
Patch by efujita on 2014-12-26: Updated 2nd patch to fix a bug in handling ExecRowMarks/ExecAuxRowMarks for foreign tables in inheritance trees
Ready for Committer Shigeru HANADA, Etsuro Fujita Etsuro Fujita, Kyotaro Horiguchi, Ashutosh Bapat 2014-12-26
Using Levenshtein distance to HINT a candidate column name
Comment by pgeoghegan on 2014-11-26: Final steps?
Patch by pgeoghegan on 2014-12-04: Revision with simplified costing logic and final, absolute suggestion quality test for best match
Patch by pgeoghegan on 2014-12-20: Revision that adds cost of distance of alias to would-be correct alias iff user specified an alias in faulty ad-hoc query (i.e. Tom and Stephen's preferred alias handling).
Needs Review Peter Geoghegan Robert Haas 2014-12-20
Add ssl_protocols configuration option
Review by ash on 2014-11-19: Review of the patch against master branch.
Patch by ash on 2014-11-27: v2: add sslprotocols libpq conn. keyword
Patch by ash on 2014-12-15: v3
Needs Review Dag-Erling Smřrgrav / University of Oslo Alex Shulgin 2014-12-15
Custom/Foreign-Join-APIs
Patch by kaigai on 2014-12-03: Initial version.
Patch by kaigai on 2015-01-06: fixup a few oversights on the v1 patch.
Needs Review KaiGai Kohei Nobody 2015-01-06
ctidscan as an example of custom-scan
Patch by kaigai on 2014-12-15: Initial version.
Comment by kaigai on 2014-12-25: module was relocated to test/modules, instead of contrib
Patch by kaigai on 2015-01-06: Several fixups according to the comment from Jim Nasby
Needs Review KaiGai Kohei Nobody 2015-01-06
parallel mode/contexts
Patch by rhaas on 2014-12-15: Initial version.
Patch by michael-kun on 2014-12-22: New patch, with improved APIs after IM discussion with Heikki
Review by andresfreund on 2015-01-04: Initial review of infrastructure
Needs Review Robert Haas & others Nobody 2015-01-04
Send server_version_num in reply to startup packet (GUC_REPORT)
Patch by ringerc on 2015-01-19: Initial version.
Needs Review Craig Ringer Nobody 2015-01-19

Performance

 
group locking
Comment by pgeoghegan on 2014-12-09: Group locking testing framework (test_group_locking-v0.1.patch)
Patch by pgeoghegan on 2014-12-10: Revised patch (group-locking-v0.1.patch), plus original explanation of testing framework (don't use this version of testing framework, though - use test_group_locking-v0.1.patch from prior link)
Comment by nmisch on 2014-12-25: Other essential thread
Needs Review Robert Haas Nobody 2014-12-25
btree_gin and ranges
Review by michael-kun on 2014-12-22: Refactoring patch for a set of macros. Main patch will need a rebase once this is pushed.
Patch by heikki on 2014-12-22: Refactoring patch pushed, here's a rebased version of the original patch.
Patch by sigaev on 2014-12-26: split patch to gin's patch and module's patch
Needs Review Teodor Sigaev Nobody 2014-12-26
Using 128-bit integers for sum, avg and statistics aggregates
Patch by kandreas on 2014-12-24: Fixes for intial review comments.
Review by davidrowley on 2014-12-26: Some questions that require some discussion. Plus benchmarks of windowing functions
Review by davidrowley on 2014-12-31: aggtransspace and aggmtransspace should only be 48 on machines that support int128. Also can you rename references to int16 to become int128?
Waiting on Author Andreas Karlsson Peter Geoghegan, David Rowley 2014-12-31
INNER JOIN removals
Patch by davidrowley on 2014-11-23: Initial version.
Patch by davidrowley on 2014-11-30: Fixes problem with PREPAREd statements
Patch by davidrowley on 2014-12-10: Changes made to allow generation of 2 plans, the correct plan is selected at execution time.
Needs Review David Rowley Nobody 2014-12-10
hashjoin - gracefully increasing NTUP_PER_BUCKET instead of batching
Patch by fuzzycz on 2014-12-07: Initial version.
Comment by fuzzycz on 2015-01-16: IMHO we should reject this, as we have no idea how to decide whether it's better than batching
Needs Review Tomas Vondra Nobody 2015-01-16
Avoiding plan disasters with LIMIT
Patch by simon on 2014-12-12: 2 patches: enable_sortedpath.v1.patch, avoid_limit_pushdown.v4.patch
Needs Review Simon Riggs Nobody 2014-12-12
Turning off HOT for larger SQL queries
Patch by simon on 2014-12-12: v7
Comment by jjanes on 2014-12-15: contrib/pgstattuple needs to be updated.
Patch by simon on 2014-12-17: v8
Needs Review Simon Riggs Nobody 2014-12-17
BRIN inclusion operator class
Patch by hasegeli on 2014-12-14: Initial version.
Review by kandreas on 2015-01-11: Initial review
Waiting on Author Emre Hasegeli Andreas Karlsson 2015-01-11

SQL Commands

 
Grouping Sets
Comment by michael-kun on 2014-12-15: Making as returned with feedback based on previous comments from Tom. This patch status was "waiting on author" either way.
Comment by rhodiumtoad on 2014-12-15: Moved to new CF and set back to "Waiting on Author"
Patch by rhodiumtoad on 2015-01-06: Updated patch.
Needs Review Andrew Gierth, Atri Sharma Tom Lane 2015-01-06
INSERT ... ON CONFLICT {UPDATE | IGNORE}
Review by pgeoghegan on 2015-01-03: Doubts emerge about the ability of any value locking implementation to maintain the useful ON CONFLICT guarantees with exclusion constraints (which might otherwise work with IGNORE).
Patch by pgeoghegan on 2015-01-11: V2.0 - Revision that adds RLS support. This closes out all open issues with support for/by interrelated features (e.g. inheritance, updatable views). Semantics now seem all but settled.
Comment by pgeoghegan on 2015-01-18: Can we cut scope, by removing the problematic exclusion constraint support for the IGNORE variant?
Needs Review Peter Geoghegan Nobody 2015-01-18
alter user/role CURRENT_USER
Review by abrightwell on 2014-10-27: Review of v2 of patch. Concerns about security and implications on items like auditing.
Patch by horiguti on 2014-11-13: I think this version takes in all the comments. Please let me know if missing something.
Patch by horiguti on 2014-11-14: V3 patch in which the new node name was changed to RoleSpec and some other chagnes made.
Needs Review Kyotaro Horiguchi Rushabh Lathia, Adam Brightwell 2014-11-14
TABLESAMPLE clause
Review by michael-kun on 2015-01-06: Patch needs a rebase, questions about structure and naming that could be improved IMO.
Patch by michael-kun on 2015-01-09: New patch, split of refactoring.
Review by michael-kun on 2015-01-09: Review of patch 1: simplification of seed generation should be done, compilation errors.
Needs Review Petr Jelinek Nobody 2015-01-09

Security

 
Row-level security
Comment by ringerc on 2014-10-17: Thread on permissive vs restrictive combinations of policies (Thom Brown)
Comment by ringerc on 2014-10-17: copy.c handling for RLS is insecure (Robert)
Comment by ringerc on 2014-10-17: WITH CHECK OPTION design concerns (Pavel)
Needs Review Many Robert Haas, Andres Freund 2014-10-17
pgaudit extension
Review by michael-kun on 2014-12-15: Review from Maumau. What's the performance impact? Temporary objects being audited is an annoyance for the lambda user.
Comment by michael-kun on 2014-12-15: Marking as returned with feedback.
Patch by ams on 2015-01-09: Updated to address Stephen's wishlist
Needs Review Ian Barwick, Abhijit Menon-Sen Nobody 2015-01-09
GetUserId() to has_privs_of_role() cleanup.
Patch by abrightwell on 2014-10-18: Initial version.
Needs Review Adam Brightwell Satoshi Nagayasu 2014-10-18

Replication & Recovery

 
Turn recovery.conf parameters into GUCs
Patch by ash on 2014-12-01: Rebased on current HEAD.
Patch by ash on 2014-12-15: v5.5
Review by pjmodos on 2014-12-23: Few issues/comments about code
Needs Review Fujii Masao, Michael Paquier, Jaime Casanova Andres Freund, Petr Jelinek 2014-12-23
Sequence Access Method
Review by heikki on 2014-11-04: Some comments on the design
Patch by pjmodos on 2014-12-12: New version - make AM fully responsible for the data in the sequence, allow AMs to specify custom columns in sequence tuple instead of single opaque amdata column
Patch by pjmodos on 2014-12-15: better ALTER SEQUENCE USING implementation
Needs Review Petr Jelinek Heikki Linnakangas 2014-12-15
Logical Replication Helper Functions
Patch by pjmodos on 2014-12-15: Initial version.
Needs Review Petr Jelinek Nobody 2014-12-15
WALWriter during Recovery
Patch by simon on 2014-12-15: Initial version.
Needs Review Simon Riggs Nobody 2014-12-15
Logical Decoding with timelines
Review by michael-kun on 2015-01-08: Heikki: Some refactoring would be nice.
Patch by michael-kun on 2015-01-08: New patch refactoring a bit the whole.
Review by michael-kun on 2015-01-08: Heikki: comments about readability
Waiting on Author Simon Riggs Nobody 2015-01-08

System Administration

 
pg_basebackup vs. Windows and tablespaces (Extend base backup to include symlink file used to restore symlinks)
Review by tgl on 2014-12-13: A few review comments
Patch by amitkapila on 2014-12-17: Addressed the review comments by Tom Lane and Robert. Still one issue is pending which is whether to allow newline in linkpath of tablespace, discussion for same is going on.
Patch by amitkapila on 2014-12-21: Fixed all the review comments raised for patch
Ready for Committer Amit Kapila dilip kumar 2014-12-21
Additional role attributes
Comment by okbobcz on 2014-10-24: waiting on new syntax
Comment by michael-kun on 2014-12-08: Patch status not updated for a while. Returned with feedback?
Patch by abrightwell on 2015-01-15: updated patch - role-attributes-v2.patch
Needs Review Stephen Frost, Adam Brightwell Pavel Stehule 2015-01-15
If pg_hba.conf has changed since last reload, emit a HINT to the client and server log
Review by petere on 2014-12-06: I think this change is problematic.
Patch by ash on 2014-12-15: v2: remove the client-side message for improved security and to avoid confusion.
Review by ssinger on 2014-12-20: The updated patch addresses the issues mentioned above.
Ready for Committer Craig Ringer Steve Singer 2014-12-20
Add restore_command_retry_interval option to control timeout of restore_command nonzero status code
Review by michael-kun on 2015-01-05: Patch unable to perform with retry intervals lower than 1s. Rewrote the patch to use TimestampTz instead, fixing at the same time issues here and there..
Patch by leopard on 2015-01-05: Michael Paquier patch
Review by andresfreund on 2015-01-16: Has wider implications than intended.
Needs Review Alexey Vasyliev Michael Paquier 2015-01-16
pg_rewind
Review by michael-kun on 2015-01-09: Documentation needs more work. Warning with MSVC compilation.
Patch by michael-kun on 2015-01-15: Patch v5.
Review by michael-kun on 2015-01-15: A couple of tiny things found on the way.
Waiting on Author Heikki Linnakangas Satoshi Nagayasu, Michael Paquier 2015-01-15
Reducing lock strength of trigger and foreign key DDL
Patch by kandreas on 2015-01-13: Fixed docs isolation tests.
Review by michael-kun on 2015-01-14: Found a couple of tiny issues. Corrected patch myself.
Patch by michael-kun on 2015-01-14: v4 of patch corrected.
Ready for Committer Andreas Karlsson Michael Paquier 2015-01-14

Monitoring & Control

 
pg_stat_ssl view
Patch by mha on 2014-11-19: Initial version.
Review by ash on 2014-12-11: Looks good, minor details spotted.
Review by michael-kun on 2014-12-22: More review comments from Heikki
Needs Review Magnus Hagander Nobody 2014-12-22

Clients

 
Allow parallel cores to be used by vacuumdb
Patch by amitkapila on 2015-01-02: cosmetic fixes.
Review by andresfreund on 2015-01-04: Not yet there.
Comment by michael-kun on 2015-01-16: Alvaro is going to have a look at it.
Needs Review Dilip Kumar Alvaro Herrera, Sawada Masahiko, Jeff Janes, Amit Kapila 2015-01-16
Do not vacuum pgbench tables if they do not exist when pgbench -f given
Patch by ishii on 2014-12-21: Initial version.
Patch by ishii on 2014-12-21: V2 patch. Check pgbench tables before vacuum. If they do not exist, silently skips vacuum.
Review by fuzzycz on 2014-12-21: slightly incorrect 'IF EXISTS' query, various nitpicking
Waiting on Author Tatsuo Ishii Nobody 2014-12-21

Procedural Languages

 
transforms
Patch by petere on 2014-12-15: Initial version.
Comment by petere on 2014-12-15: see also previous entry: https://commitfest.postgresql.org/action/patch_view?id=1067
Review by michael-kun on 2014-12-22: Little comment from Tom.
Needs Review Peter Eisentraut Nobody 2014-12-22

Miscellaneous

 
adaptive ndistinct estimator
Patch by fuzzycz on 2014-11-23: Initial version.
Patch by fuzzycz on 2014-12-07: v3 of the patch, implementing the 'minimal' case and various other improvements
Needs Review Tomas Vondra Nobody 2014-12-07
decreasing array_agg memory consumption
Comment by fuzzycz on 2015-01-07: marked as 'needs review' as the status was obsolete
Comment by apaan on 2015-01-08: element checking, behavior of makeArrayResult if release=true while subcontext=false
Patch by fuzzycz on 2015-01-16: v8 - added/fixed a few comments, agree with marking as ready for committer
Needs Review Tomas Vondra Ali Akbar 2015-01-16
moving from contrib to bin
Comment by michael-kun on 2014-12-22: Patch for the MSVC portion
Patch by michael-kun on 2014-12-23: New set of patches: moved utility pages to ref/ and removed references to authors, with reworked patch for MSVC
Review by michael-kun on 2014-12-24: Previous message is as well a review btw..
Waiting on Author Peter Eisentraut Michael Paquier 2014-12-24

Bug Fixes

 
Error when creating names too long for tar format
Patch by petere on 2014-12-04: Initial version.
Comment by petere on 2014-12-04: Part of this ought to be backpatched. Part of it depends on the outcome of <https://commitfest.postgresql.org/action/patch_view?id=1512>.
Needs Review Peter Eisentraut Nobody 2014-12-04
Perl coding error in msvc build system
Patch by petere on 2014-12-04: Initial version.
Needs Review Peter Eisentraut Nobody 2014-12-04
Track TRUNCATE via pgstat: reset live/dead tuple counters
Comment by decibel on 2014-12-15: Not sure how patch was created, but git apply and am couldn't handle it. Ended up using patch -p1 --dry-run <~/Downloads/truncate-and-pgstat-v0.2.patch
Review by decibel on 2014-12-16: Review + test patch
Patch by michael-kun on 2014-12-22: Patch v0.3, addressing problem with 2PC
Needs Review Alex Shulgin Jim Nasby 2014-12-22

Code Comments

 
Clarify need for memory barriers in bgworkers
Patch by decibel on 2014-11-06: Initial version.
Needs Review Jim Nasby Nobody 2014-11-06

Documentation

 
documentation update for doc/src/sgml/func.sgml
Patch by ads on 2014-10-17: New version, includes changes requested by Peter
Patch by michael-kun on 2014-12-22: Doc format is still incorrect, new patch.
Patch by calvin on 2014-12-31: Treat type names homogeneously.
Ready for Committer Andreas Scherbaum, Josh Berkus Fabien Coelho 2014-12-31

Committed Patches

Patch Name Status Author Reviewers Date Closed
Fix silly coding in pgcrypto
Patch by johto on 2014-11-02: Initial version.
Committed Marko Tiikkaja Nobody 2014-11-03
Documentation fixes for BRIN indexes
Patch by michael-kun on 2014-11-26: Initial version.
Committed Michael Paquier Nobody 2014-12-02
Spurious set in heap_prune_chain
Patch by decibel on 2014-11-06: Initial version.
Committed Jim Nasby Simon Riggs 2014-12-08
Point to polygon distance operator
Patch by hasegeli on 2014-09-14: Initial version.
Patch by hasegeli on 2014-09-14: Split patch
Committed Alexander Korotkov Emre Hasegeli 2014-12-15
Enable detection of c++ files in MSVC scripts
Patch by michael-kun on 2014-11-26: Initial version.
Patch by michael-kun on 2014-11-27: Rewrite grammar of patch and take into account lpp and ypp files.
Committed Michael Paquier Muhammad Asif Naeem 2014-12-17
speedup tidbitmap patch: hash BlockNumber
Patch by sigaev on 2014-12-17: Remove wrong comment
Review by tgl on 2014-12-17: Still needs work, also maybe we need more thought about how to change hash_create exactly.
Patch by tgl on 2014-12-18: Revised patch here.
Committed Teodor Sigaev Heikki Linnakangas 2014-12-18
Log inabality to acquire cleanup lock in vacuum
Patch by decibel on 2014-11-06: Initial version.
Patch by decibel on 2014-12-01: New patch only logs; no retry.
Committed Jim Nasby Heikki, Robert, Andres 2014-12-18
Replace Red-Black tree with Pairing Heap in knn-GiST search
Comment by jjanes on 2014-12-15: undefined function pairingheap_empty when under --enable-cassert
Patch by heikki on 2014-12-17: Latest patch version
Review by pgeoghegan on 2014-12-20: Minor observations on style, necessity of remove function.
Committed Heikki Linnakangas Peter Geoghegan 2014-12-22
Some bug fixes for REINDEX SCHEMA: unused variables defined, matviews and regression tests
Comment by michael-kun on 2014-12-11: Both patches should be committed.
Review by michael-kun on 2014-12-11: Patch works as expected. Those are only things bugs missing in former implementation.
Comment by michael-kun on 2014-12-15: Patch to fix regressions and matviews pushed, still missing the removal of do_user and do_database in ReindexStmt.
Committed Sawada Masahiko, Michael Paquier Fabrízio Mello 2014-12-24
event triggers: more DROP info
Patch by alvherre on 2014-10-15: Initial version.
Committed Álvaro Herrera Nobody 2014-12-30
Fix xpath() to return namespace definitions (fixes the issue with nested or repeated xpath())
Patch by apaan on 2014-11-26: patch v7: rebase to master
Comment by michael-kun on 2014-12-15: Patch needs more work, per the author, hence switching to returned with feedback.
Comment by apaan on 2014-12-15: the v7 patch is good as it is. Will try to reduce performance regression in another patch.
Committed Ali Akbar Peter Eisentraut 2015-01-07
Print sort ordering options in EXPLAIN
Comment by maiku41 on 2015-01-14: Looks good other than quoting issue with collation name.
Waiting for doc and test update.
Patch by mtimm_01 on 2015-01-15: Version 8, fixing remaining issues (e.g. quoting), adding docs and tests as requested/agreed.
Review by maiku41 on 2015-01-15: Remaining issues addressed. Looks good to me. Marking as ready for committer.
Committed Marius Timmer, Lukas Kreft, Arne Scheffer Mike Blackwell 2015-01-16
speedup tidbitmap patch: cache page
Patch by sigaev on 2014-12-17: Fix bugs introduced in v2.1
Review by davidrowley on 2014-12-21: Small suggestion and benchmark results, plus patch which improves the comments a little. We need to decide which version to use.
Review by davidrowley on 2014-12-24: Looks ok to me. I think v3 is the one to go for.
Committed Teodor Sigaev David Rowley 2015-01-16
advance local xmin more aggressively
Patch by rhaas on 2014-12-15: Competing version from Heikki.
Review by michael-kun on 2014-12-22: Heikki's patch: Missing semi-colon and patch does not build with cassert
Patch by heikki on 2014-12-23: Fixed and rebased version of Heikki's patch
Committed Robert Haas vs. Heikki Linnakangas Nobody 2015-01-16
Abbreviated keys (Was: Sort support for text with strxfrm() poor man's keys)
Patch by pgeoghegan on 2014-10-12: Patch that integrates abbreviated keys with a quasi-independent effort to make the B-Tree and CLUSTER cases use sortsupport. Similar benefits (and costs) to the MinimalTuple case.
Patch by pgeoghegan on 2014-11-10: Rebased patch
Patch by pgeoghegan on 2014-12-03: Revision with minor tweaks requested by Robert
Committed Peter Geoghegan Robert Haas 2015-01-19
Fix installation of shared libraries on Windows (mingw)
Comment by petere on 2014-12-05: Previously attached to this entry: https://commitfest.postgresql.org/action/patch_view?id=1330, but made a separate entry now.
Review by michael-kun on 2014-12-22: Should include fix for MSVC that got lost in the void.
Patch by michael-kun on 2014-12-24: Patch for MSVC portion included as well for consistency with existing patch. Would be good to get that backpatched.
Committed Peter Eisentraut Michael Paquier 2015-01-19

Returned with Feedback

Patch Name Status Author Reviewers Date Closed
Compression of Full Page Writes
Patch by michael-kun on 2014-12-28: New patch set, make pglz an entirely independent frontend/backend facility and rebase core patch on that.
Review by michael-kun on 2015-01-07: Some review from Rahila: concerns about readability, issue with pg_xlogdump and some incorrect comments.
Patch by michael-kun on 2015-01-07: Updated patch addressing previous comments.
Returned with Feedback Rahila Syed, Michael Paquier Abhijit Menon-Sen 2015-01-15
compress method for spgist
Review by heikki on 2014-12-15: Review. Needs documentation.
Patch by sigaev on 2014-12-16: Third version, add docs
Review by heikki on 2014-12-23: Confused on reconstructedValue's datatype
Returned with Feedback Teodor Sigaev Heikki Linnakangas 2015-01-15
Allow TOAST tables to be moved to a different tablespace
Patch by ash on 2014-12-05: Initial version.
Review by kandreas on 2014-12-30: Initial review with feedback.
Returned with Feedback Julien Tachoires, Alex Shulgin Andreas Karlsson 2015-01-15
Join pushdown support for foreign tables
Patch by hanada on 2014-12-15: Initial version.
Comment by kaigai on 2014-12-19: some brief comments
Comment by kaigai on 2014-12-19: suggestion to use fdw_ps_tlist to map underlying var-node on pseudo target-list
Returned with Feedback Shigeru HANADA KaiGai Kohei 2015-01-15
logical column ordering (WIP)
Patch by alvherre on 2014-12-15: Initial version.
Comment by andresfreund on 2015-01-04: Alvaro announced reworking the patch.
Returned with Feedback Álvaro Herrera Nobody 2015-01-15
UPDATE table SET(*)=
Patch by atris on 2014-10-15: Initial version.
Patch by johto on 2014-10-29: Initial review
Review by michael-kun on 2014-12-22: Complaint from Tom about using transformTargetList.
Returned with Feedback Atri Sharma Marko Tiikkaja 2015-01-15
DROP PRIVILEGES OWNED
Patch by johto on 2014-12-15: Initial version.
Review by michael-kun on 2014-12-22: Comment from Heikki: change grammar from DROP to REVOKE as it seems to be more adapted
Returned with Feedback Marko Tiikkaja Nobody 2015-01-15
multivariate statistics
Review by davidrowley on 2014-10-29: Needs modified to support C89
Patch by fuzzycz on 2014-11-10: v2 of the patch (with the C89 errors fixed), however, there's some mixup in varno handling
Review by michael-kun on 2014-12-22: Comments from Heikki
Returned with Feedback Tomas Vondra Nobody 2015-01-15
HINT on qualified LHS references in UPDATE .. SET
Patch by johto on 2014-11-22: Initial version.
Patch by johto on 2014-11-22: Patch v2: fix behaviour with aliases
Review by johto on 2014-11-23: Tom's review
Returned with Feedback Marko Tiikkaja Peter Geoghegan 2015-01-15
Async execution of postgres_fdw.
Patch by michael-kun on 2015-01-15: v2 of patch
Patch by michael-kun on 2015-01-15: v3 of patch
Comment by michael-kun on 2015-01-16: Work still on the way and cannot obviously be committed for this CF, hence moving to next one.
Returned with Feedback Kyotaro Horiguchi Ashutosh Bapat 2015-01-16
Memory-bounded HashAgg
Comment by jdavis on 2014-12-28: More tests
Review by fuzzycz on 2015-01-07: patch measures hashcontext, not aggcontext (as it should), fix results in significant performance hit
Comment by michael-kun on 2015-01-16: Marking as returned with feedback, patch is incorrect and once corrected performance drops significantly.
Returned with Feedback Jeff Davis Nobody 2015-01-16
Memory accounting towards memory-bounded HashAgg
Patch by michael-kun on 2014-12-08: New version.
Review by michael-kun on 2014-12-22: Review from Tomas: crashes found and regression tests lacking
Comment by jdavis on 2014-12-23: New HashAgg patch. Memory accounting patch is unchanged.
Returned with Feedback Jeff Davis Peter Geoghegan, Tomas Vondra 2015-01-19

Rejected Patches

Patch Name Status Author Reviewers Date Closed
Escaping from blocked send() by pg_terminate_backend().
Review by michael-kun on 2014-12-15: Review of Heikki for patch 3.
Review by michael-kun on 2014-12-15: Concerns about current approach.
Comment by horiguti on 2015-01-16: This patch became a part of "Overhauling our interrupt handling".
Rejected Kyotaro Horiguchi Nobody 2014-12-15
Role Attribute Bitmask Representation
Patch by abrightwell on 2014-12-08: Updated patch. role-attribute-bitmask-v4.patch - does not include documentation updates.
Patch by abrightwell on 2014-12-15: Updated patch. role-attribute-bitmask-v5.patch - includes documentation updates.
Patch by abrightwell on 2014-12-20: Updated patch. role-attribute-bitmask-v7.patch - includes documentation updates and proposed changes from Alvaro Herrera.
Rejected Adam Brightwell Álvaro Herrera 2014-12-23
Fractions in GUC variables
Patch by johngorman on 2014-12-08: Initial version.
Review by petere on 2014-12-15: Not sure we want this (anymore).
Rejected John Gorman Peter Eisentraut 2014-12-24
Comment on "lack" of optimization in CacheInvalidateHeapTuple
Patch by decibel on 2014-11-06: Initial version.
Rejected Jim Nasby Nobody 2015-01-04
Fillfactor for GIN indexes
Patch by michael-kun on 2014-11-21: Initial version.
Review by smagen on 2014-11-28: First look at the patch.
Patch by michael-kun on 2015-01-07: Updated patch. Default fillfactor value to 75% is good? This increases default size of index..
Rejected Michael Paquier Alexander Korotkov 2015-01-16