From: | Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: SonarQube Fixes #4 Database & Foreign Tables |
Date: | 2021-01-21 11:54:01 |
Message-ID: | CAKtn9dN2O-g-PwKT8qYaAqMS3ohkZ76JpvBPSR2_qnQ7Tjsarg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Thu, Jan 21, 2021 at 2:47 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Rahul
>
> Following are the review comments:
>
> - Change the name of the function 'get_pg_db_properties'. The function
> returns lastsysoid and datistemplate only and I assume it is applicable for
> PG/EPAS both.
>
> *Done!*
>
> - Add comments for all the new functions introduce in the patch.
>
> *Added!*
>
> On Thu, Jan 21, 2021 at 2:36 PM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Please find the attached patch which resolves the sonar qube issues
>> relating:
>>
>>
>> 1. *Foreign Tables* -*Refactor this function to reduce its Cognitive
>> Complexity from 67 to the 15 allowed.*
>> 2. *Database* -*Refactor this function to reduce its Cognitive
>> Complexity from 17 to the 15 allowed.*
>>
>>
>> --
>> *Rahul Shirsat*
>> Senior Software Engineer | EnterpriseDB Corporation.
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
--
*Rahul Shirsat*
Senior Software Engineer | EnterpriseDB Corporation.
Attachment | Content-Type | Size |
---|---|---|
sonar_issues_database_foreign_tables_v2.patch | application/octet-stream | 11.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Nidhi Bhammar | 2021-01-21 14:36:36 | [pgAdmin][RM6147] - Kerberos documentation |
Previous Message | Aditya Toshniwal | 2021-01-21 11:02:51 | Re: [pgAdmin][RM1802] ERD Tool (Beta) |