From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Per-column collation, the finale |
Date: | 2011-02-04 21:24:19 |
Message-ID: | 201102042124.p14LOJf23714@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Noah Misch wrote:
> On Thu, Feb 03, 2011 at 05:53:28PM +0200, Peter Eisentraut wrote:
> > On tor, 2011-02-03 at 00:10 -0500, Noah Misch wrote:
> > > > This is good stuff. I'll send you a new patch in a day or three for
> > > > perhaps another round of performance tests. Some of the other
> > > issues
> > > > above can perhaps be postponed for follow-up patches.
> > >
> > > I agree -- if the performance-when-unused gets solid, none of my other
> > > comments ought to hold things up.
> >
> > Here is a new patch.
> >
> > The main change is in varstr_cmp(), avoiding the calls to
> > pg_newlocale_from_collation() when the default locale is used. This
> > accounts for the performance regression in my tests. It also addresses
> > some of your refactoring ideas.
>
> Looks good and tests well. I've attached the same benchmark script with updated
> timings, and I've marked the patch Ready for Committer.
Nice to see that performance hit is removed now!
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2011-02-04 21:53:17 | Re: is_absolute_path incorrect on Windows |
Previous Message | Dimitri Fontaine | 2011-02-04 20:55:41 | Re: Extensions support for pg_dump, patch v27 |