Re: [pgAdmin][RM4988] Refactoring of all the SQL files used by pgAdmin4

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Cc: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Subject: Re: [pgAdmin][RM4988] Refactoring of all the SQL files used by pgAdmin4
Date: 2019-12-18 07:42:04
Message-ID: CAM9w-_nwTMzSpDzp4Xb=8unmrzDD3-anma88AtcAiNNtCuk6XA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Hackers,

Had couple of typos in the previous patch. Although the edit types were
passed to the front end, they were not filtered and all the types were
displayed in edit table dialog.
Kindly find the patch to fix it.

On Mon, Dec 16, 2019 at 6:54 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Thanks, patch applied.
>
> On Mon, Dec 16, 2019 at 5:40 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Attached patch which is related to RM4988, reduces the number of hits to edit_mode_types.sql
>> (an expensive one).
>> Currently, this is query is executed for every column one by one when
>> table properties dialog is opened. I've added edit_mode_types_bulk.sql,
>> which will fetch the edit types for all the distinct column types used in
>> the table in a single DB trip and use that data for each column.
>> Also, while generating RE-SQL for columns, edit_mode_types.sql will not
>> be executed.
>> This will definitely reduce time.
>>
>> Kindly review.
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> Sr. Software Engineer | EnterpriseDB India | Pune
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>

--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"

Attachment Content-Type Size
RM4988.edit_types.typos.patch application/octet-stream 1.0 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2019-12-18 08:06:18 pgAdmin 4 commit: Fixed couple of typos for edit types which results in
Previous Message Akshay Joshi 2019-12-17 11:58:02 pgAdmin 4 commit: Updated node alpine image from 8 to 10.