From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Upgrading our minimum required flex version for 8.5 |
Date: | 2009-07-12 21:43:47 |
Message-ID: | 23844.1247435027@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> If we're going to have a reentrant lexer, I think we should go the whole
> nine yards. I agree that a couple of percent slowdown on just the lexing
> and parsing will be lost in the noise. So +1 from me.
I found a couple of places where a few cycles could be shaved. The
current version of the patch (attached) seems indistinguishable in
speed from 8.4 on my HPPA box, though still a percent or two slower on
my x86_64 box.
This is ready to go except for changing the minimum flex version test
in configure (and associated documentation). I see that a good-sized
fraction of the buildfarm is still on flex 2.5.4 and will therefore go
red when this goes in. Should I hold off a bit longer, or is committing
the best way to get their attention?
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
reentrant-parser-1.patch.gz | application/octet-stream | 15.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ron Mayer | 2009-07-12 21:59:22 | Re: Maintenance Policy? |
Previous Message | Robert Haas | 2009-07-12 21:15:53 | Re: Logging configuration changes |