i believe the last submission of the patch had no negative performance impact on any of the tested use cases, but you'd have to go back and see the last exchange on that.
i think it was the concern about potentially regressing the codeline in unforeseen ways without a clear benefit to all but one use case (wide tables) that stalled things. >________________________________ > From: Josh Berkus <[email protected]> >To: Simon Riggs <[email protected]> >Cc: Tom Lane <[email protected]>; Amit Kapila <[email protected]>; >Andres Freund <[email protected]>; Craig Ringer <[email protected]>; >Jameison Martin <[email protected]>; Noah Misch <[email protected]>; Kevin >Grittner <[email protected]>; [email protected]; >[email protected] >Sent: Monday, June 24, 2013 10:32 PM >Subject: Re: [HACKERS] patch submission: truncate trailing nulls from heap >rows to reduce the size of the null bitmap [Review] > > >Simon, > >> I think its rather a shame that the proponents of this patch have >> tried so hard to push this through without working variations on the >> theme. Please guys, go away and get creative; rethink the approach so >> that you can have your lunch without anybody else losing theirs. I >> would add that I have seen the use case and want to support it, but >> we're looking to add to the codebase not just steal small bites of >> performance from existing use cases. > >Do we really have ironclad numbers which show that the patch affects >performance negatively? I didn't think the previous performance test >was comprehensive; I was planning to develop one myself this week. > >-- >Josh Berkus >PostgreSQL Experts Inc. >http://pgexperts.com > > >
