Log In - Home Page

Patch: Trim trailing NULL columns

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2012-11
Topic Performance
Patch Status Returned with Feedback
Author Jameison Martin
Reviewers Josh Berkus, Amit Kapila
Committer Greg Stark
Close Date 2013-01-11
Comments
Patch by aglio on 2012-05-03 06:58:40 PM: Initial version.
Comment by aglio on 2012-05-03 07:00:00 PM: Testing information given.
Comment by rhaas on 2012-06-26 09:05:11 PM: Ping.
Comment by amitkapila on 2012-10-14 04:27:02 AM: performance data which shows, it is a win when there are large number of columns having NULL values. it also shows there is no loss for non-NULL values.
Review by amitkapila on 2012-10-16 04:24:49 AM: few review comments needs to be addressed.
Patch by amitkapila on 2012-10-19 12:12:57 PM: updated patch to handle review comments.
Comment by simon on 2013-01-11 11:00:46 AM: Performance tests show a reasonable gain in the extreme cases, but they are sensitive to exact layout of values. Performance regression of 1% on normal pgbench. Impact > benefit, so rejected from this CF. More work is possible in future.

Add Comment

Please log in to comment on this patch.