On Thu, Jan 21, 2021 at 2:47 PM Akshay Joshi <akshay.jo...@enterprisedb.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.shir...@enterprisedb.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.
sonar_issues_database_foreign_tables_v2.patch
Description: Binary data
