Log In - Home Page

Patch: Inverse Aggregate Transition Functions

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2014-01
Topic Performance
Patch Status Committed
Author David Rowley, Florian Pflug
Reviewers Dean Rasheed
Committer Tom Lane
Close Date 2014-04-12
Patch by davidrowley on 2013-12-27 05:56:45 AM: Initial version.
Comment by petere on 2014-01-11 07:18:42 PM: build failed: http://pgci.eisentraut.org/jenkins/job/postgresql_commitfest_world/301/
Comment by petere on 2014-01-11 07:18:53 PM: There are more recent patches in the thread.
Patch by davidrowley on 2014-01-11 11:11:13 PM: Updated patch
Patch by deanr on 2014-01-24 08:58:43 AM: Latest patch from the authors
Review by deanr on 2014-01-24 09:00:03 AM: Suggest breaking the patch up. Problem with FILTERed rows and non-strict transition functions.
Patch by fgp on 2014-01-25 09:18:14 AM: Split patch into 5 parts, fixed FILTER bug, included all pending stuff I had in my tree but not yet posted. Ready for review.
Patch by fgp on 2014-02-20 01:55:25 AM: Require strictness of fwd and inv transfun to match, docs not yet updated
Patch by fgp on 2014-02-21 11:56:20 AM: Rebased to apply to HEAD again
Review by deanr on 2014-03-03 08:48:16 AM: Some review comments for the base patch
Patch by fgp on 2014-03-04 02:15:28 AM: Incorporated most changes suggested by Dean's review of the base patch
Patch by fgp on 2014-03-31 01:00:11 AM: Simplified EXPLAIN ANALYZE VERBOSE support and updated docs
Patch by fgp on 2014-04-01 09:06:18 PM: Rebased to HEAD (554bb3beba27bf4a49edecc40f6c0f249974bc7c)
Patch by fgp on 2014-04-09 02:06:00 AM: Included Dean Rasheed's performance optimization. Put bool aggs into separate patch. MIN/MAX stuff has risk of performance regression, shouldn't be applied I think.
Comment by deanr on 2014-04-09 08:10:30 AM: Believed to be ready for committer, with the exception of the min/max aggregates patch.

Add Comment

Please log in to comment on this patch.