| From: | Stephen Frost <sfrost(at)snowman(dot)net> | 
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org | 
| Subject: | Re: Slow tab completion w/ lots of tables | 
| Date: | 2012-08-21 17:48:48 | 
| Message-ID: | 20120821174847.GL1267@tamriel.snowman.net | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> The LIKE idea is interesting.  What you'd still need is to suppress the
> quote_ident function call so that it becomes just "relname LIKE 'foo%'".
> Which seems do-able if possibly rather ugly.  That would leave us with
> SELECT ... FROM foo<TAB> being fast but SELECT ... FROM "foo<TAB> being
> slow; not sure if that's good enough.  I do *not* want to add an index
> on quote_ident(relname) to pg_class though.  Quite aside from speed
> issues, I'm pretty sure that functional indexes on core system catalogs
> would be a huge can of worms.
That's the kind of concern that I was expecting, to be honest. :)  As
Kevin's pointed out, it's not likely to be needed anyway..  There's a
bit of an open question still regarding case-insensitive searching, but
perhaps we let that be slow and only done if we don't get any answers
back from a case-sensetive search?
For example, given these tables:
TEST
test
Doing:
select * from TE<tab>  -> "TEST"
select * from "TE<tab>  -> "TEST"
select * from te<tab>  -> test
select * from "te<tab>  -> test
select * from Te<tab>  -> test (but slow)
select * from "Te<tab>  -> test (but slow)
We'd essentially do: LIKE 'xx%', and then run quote_ident() on the
result (I assume we can replace the whole word, right?).  I'd also
strip off any ", for the purposes of searching with tab-completion.  I'm
not sure how easy it'd be to have a fall-back setup.  I do wonder if we
should do what I often recommend my dev do though, which is to have a
SQL or pl/pgsql function defined on the database-side, rather than
sending large/complex queries to the database from the application..
Thanks,
		Stephen
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amit kapila | 2012-08-21 17:51:16 | Re: 9.2RC1 wraps this Thursday ... | 
| Previous Message | Tom Lane | 2012-08-21 17:37:16 | Re: Slow tab completion w/ lots of tables |