Log In - Home Page

Patch: Remove unused targets from plan (from 2013-01)

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2013-06
Topic Performance
Patch Status Returned with Feedback
Author Alexander Korotkov, Etsuro Fujita
Reviewers Hitoshi Harada, Álvaro Herrera
Committer Tom Lane
Close Date 2013-08-02
Comments
Patch by smagen on 2012-11-12 10:26:05 AM: Initial version.
Review by efujita on 2012-12-04 06:02:40 AM: Initial review on the patch
Comment by efujita on 2012-12-04 06:07:28 AM: Comments by Tom
Comment by ringerc on 2013-01-18 10:54:44 AM: Pending revision by author
Comment by ringerc on 2013-03-03 02:40:38 PM: Bumped to next CF at author request.
Patch by efujita on 2013-04-08 09:57:51 AM: Updated version using a planner-based approach
Patch by efujita on 2013-06-18 07:32:16 AM: Rebased against the latest head.
Patch by efujita on 2013-06-18 12:14:42 PM: Cleanup
Review by umitanuki on 2013-06-19 05:58:08 AM: Window function is not covered.
Patch by efujita on 2013-06-20 07:26:07 AM: Fixed the problem reported by Harada-san.
Review by umitanuki on 2013-06-21 10:11:49 AM: Looks good to me.
Patch by efujita on 2013-06-21 10:45:29 AM: Fixed the comment typo.
Review by alvherre on 2013-06-25 04:11:50 PM: please add a few comments
Patch by efujita on 2013-07-03 06:43:12 AM: Added some comments and improved the code a little bit.
Review by tgl on 2013-08-02 10:57:03 PM: Should take the cost savings into account while choosing the plan, not just whack the tlist around afterwards...

Add Comment

Please log in to comment on this patch.