From: | Edmund Horner <ejrh00(at)gmail(dot)com> |
---|---|
To: | "C,C H" <a12321aabb(at)gmail(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Add a tab-completion for "SELECT <sth> INTO" or "SELECT <sth> FROM" in psql |
Date: | 2018-07-02 09:19:24 |
Message-ID: | CAMyN-kCw-9Cv3tCibikvko8ek+H4zjBSiW0wk8AT2BWNJKDA0Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2 July 2018 at 17:57, C,C H <a12321aabb(at)gmail(dot)com> wrote:
> I use tab-completion in psql quite often and I find that I can't complete
> "FROM" for SELECT query.
>
> So I try to create a patch for it.
>
> I download the source code from GitHub master branch and modify the file to
> create the patch.
>
> I compile the code and my code change works like follows,
> --------------------------------------------------------------
> postgres=# SELECT * <Tab Key><Tab Key>
> FROM INTO
> postgres=# SELECT *
> --------------------------------------------------------------
>
>
> This is my first patch contribution trial here, please correct me if there
> is something wrong.
Hi CCHsu,
This looks useful, but it does overlap with my patch to add completion
for column and function names to SELECT -- see
https://commitfest.postgresql.org/18/1376/ . I have been quiet on
this as everyone was (and still is) busy with the 11.0 release.
I also have a patch for columns and functions after a comma in the
select list, but I've been even more quiet on that one.
We can try to combine them. It won't be as simple as your patch here,
unfortunately. Luckily the completions are active at mostly*
different times. Was the last word SELECT ? Offer a column or
function name. Otherwise offer FROM/INTO.
* note that FROM/INTO can also appear directly after SELECT!
Cheers,
Edmund
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2018-07-02 09:22:46 | Re: Threat models for DB cryptography (Re: [Proposal] Table-level Transparent Data Encryption (TDE) and Key) Management Service (KMS) |
Previous Message | Magnus Hagander | 2018-07-02 08:45:02 | Re: branches_of_interest.txt |