| From: | Edmund Horner <ejrh00(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Vik Fearing <vik(dot)fearing(at)2ndquadrant(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: PATCH: psql tab completion for SELECT |
| Date: | 2018-03-21 06:51:52 |
| Message-ID: | CAMyN-kA3m2ApH6Q6TmpYMtnq7t7DsQGdGo4JPAE7BSz6GX80xQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 8 March 2018 at 17:23, Edmund Horner <ejrh00(at)gmail(dot)com> wrote:
> New patch that fixes a little bug with appending NULL addons to schema queries.
Hi all, I haven't heard anything for a while and so assume you're
beavering away on real features. :)
I've been dogfooding this patch at work, and I am personally pretty
happy with it.
I still think the number of completions on an empty string is a bit
too big, but I don't know what to omit. There are around 1700
completions on the empty "postgres" database in my testing, and we
show the first 1000 (arbitrarily limited, as the generated SQL has
LIMIT 1000 but no ORDER BY).
Should we just leave it as is?
Whether we improve the filtering or not, I'm optimistic the feature
will be committed in this CF or the next.
I've also been working on adding support for completions after commas,
but I really want to see the current feature finished first.
Cheers,
Edmund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2018-03-21 07:06:49 | Re: JIT compiling with LLVM v12.2 |
| Previous Message | Tom Lane | 2018-03-21 05:51:01 | Re: [HACKERS] taking stdbool.h into use |