From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Joerg Sonnenberger <joerg(at)bec(dot)de> |
Cc: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>, John Naylor <jcnaylor(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: reducing the footprint of ScanKeyword (was Re: Large writable variables) |
Date: | 2019-01-07 21:11:04 |
Message-ID: | 11173.1546895464@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I wrote:
> I took a quick look through the NetBSD nbperf sources at
> http://cvsweb.netbsd.org/bsdweb.cgi/src/usr.bin/nbperf/
> and I concur with your judgment that we could manage translating
> that into Perl, especially if we only implement the parts we need.
Here's an implementation of that, using the hash functions you showed
upthread. The speed of the Perl script seems to be pretty acceptable;
less than 100ms to handle the main SQL keyword list, on my machine.
Yeah, the C version might be less than 1ms, but I don't think that
we need to put up with non-Perl build tooling for that.
Using the same test case as before (parsing information_schema.sql),
I get runtimes around 3560 ms, a shade better than my jury-rigged
prototype.
Probably there's a lot to be criticized about the Perl style below;
anybody feel a need to rewrite it?
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
perfect-hash-keyword-lookup-1.patch | text/x-diff | 20.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | didier | 2019-01-07 21:44:24 | Re: [proposal] Add an option for returning SQLSTATE in psql error message |
Previous Message | John Naylor | 2019-01-07 21:00:14 | Re: reducing the footprint of ScanKeyword (was Re: Large writable variables) |