Re: Regarding codemirror upgrade and query tool autocomplete

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Regarding codemirror upgrade and query tool autocomplete
Date: 2024-01-04 06:05:53
Message-ID: CANxoLDcRZVGSeFdvtVYdXDOMEFTvV5bZnp4scmcDbtcVY=uBxA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Aditya

On Tue, Jan 2, 2024 at 6:33 PM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> Hi,
> Just to add to my previous email, double quoted table autocomplete works
> now with the latest minor version. More reasons to use the new approach.
>

I think we should use the new approach.

>
> On Tue, Jan 2, 2024 at 4:44 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> I am working on upgrading the CodeMirror package (used by SQL input/Query
>> editor) to the latest version. One major improvement is on the autocomplete
>> part. CodeMirror provides a very versatile way of using autocomplete now.
>> One needs to pass the schema, table and columns data and it will show up in
>> intellisense on typing.
>> Right now, we send the query to the backend and the backend sends the
>> suggestions which are shown on the screen. This can be very slow for remote
>> PostgreSQL servers as it queries everytime to the server for data and
>> requires an active connection. Also, not a good user experience.
>> I was playing around with the new autocomplete and I have attached the
>> screen-recording for the same. The data from the backend is loaded once the
>> query tool opens.
>> It has a few issues though. Column suggestions don't work correctly for
>> double quoted tables aliases. It doesn't suggest "joins" which we do right
>> now.
>> I still need to figure out how we can refresh the data when new
>> tables/columns are added/removed. Overall, I find it more useful to the
>> users as it is quick and does the job what someone expects from an
>> autocomplete/intellisense.
>>
>> Thoughts/Suggestions on whether we should use it or continue with
>> existing?
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin Hacker | Sr. Software Architect | *enterprisedb.com*
>> <https://www.enterprisedb.com/>
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> Thanks,
> Aditya Toshniwal
> pgAdmin Hacker | Sr. Software Architect | *enterprisedb.com*
> <https://www.enterprisedb.com/>
> "Don't Complain about Heat, Plant a TREE"
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message dependabot[bot] 2024-01-04 10:32:42 [pgadmin-org/pgadmin4] 1bebf8: Javascript dependency: Bump axios from 1.6.0 to 1....
Previous Message Khushboo Vashi 2024-01-03 10:39:54 [pgadmin-org/pgadmin4] fdaeda: Fixed multiple CLI issues. #2483