Re: [pgAdmin][RM5666]: Added missing dependencies/dependent node - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin][RM5666]: Added missing dependencies/dependent node
Date
Msg-id CAM9w-_=PeLkF-uaasR9-XvhwRpypJs9QA+W7VT2MYyb+VuYNzQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][RM5666]: Added missing dependencies/dependent node  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Responses Re: [pgAdmin][RM5666]: Added missing dependencies/dependent node  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Hi Akshay,

The partition of a table is showing an extra "Partition index" dependents, whereas there are no nodes under indexes node.
The primary key displayed twice is an existing issue, confirmed with 4.23.
Screenshot 2020-07-14 at 4.02.23 PM.png

Apart from that, the patch looks good to me. Test cases success on all the server versions.

On Tue, Jul 14, 2020 at 1:15 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi,

I'll be reviewing the patch.

On Tue, Jul 14, 2020 at 12:46 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Hackers

Attached is the patch to add missing dependencies/dependent nodes

Added following missing nodes:
  • Materialized View
  • Event Trigger
  • Collation
  • FTS Configuration, FTS Dictionary, FTS Parser, FTS Template,
  • RLS Policy
  • Extension
  • Synonyms (EPAS)
  • Packages (EPAS)
  • Compound Trigger (EPAS)
Following bugs fixed with this patch: 
  • Foreign Table is interpreted as a Foreign Data Wrapper.
  • Domain is interpreted as a type.
  • Domain constraint is interpreted as a check constraint. 
  • Partitioned Table is interpreted as a Partition.
  • Trigger function is interpreted as a function in the dependencies tab for Event Triggers.
  • Procedure interpreted as a Function.
Please review it.

--
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"


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

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][SonarQube] Duplicate CSS blocks
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch] - SonarQube Fixes