Re: [pgAdmin4][Patch]: #3412 - Add support for PRIMARY KEY, FOREIGNKEY, indexes, and triggers on partitioned tables for PG 11 - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [pgAdmin4][Patch]: #3412 - Add support for PRIMARY KEY, FOREIGNKEY, indexes, and triggers on partitioned tables for PG 11
Date
Msg-id CANxoLDc1RbqdMJ2WK9k6vN=k+=_HddYx=xkQte0OJXzZ2-=wkw@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][Patch]: #3412 - Add support for PRIMARY KEY, FOREIGN KEY,indexes, and triggers on partitioned tables for PG 11  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch]: #3412 - Add support for PRIMARY KEY, FOREIGNKEY, indexes, and triggers on partitioned tables for PG 11
List pgadmin-hackers
Hi Khushboo

Following are my initial review comments:
  • Fix PEP-8 warnings.
  • We should hide "Modulus" and "Remainder" columns for PG/EPAS 10 as Hash partition is not supported in that.
  • Primary key switch control on "columns" tab should be enabled for partitioned table created on PG/EPAS 11.
  • Please verify that Unique constraint are supported for PG/EPAS 11 or not. 
Will do code review tomorrow.

On Wed, Jul 25, 2018 at 4:29 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,


Please find the attached patch for RM #3412.

- Add support for PRIMARY KEY, FOREIGN KEY, indexes, and triggers on partitioned tables for PG 11

- Add support for partitioning by  HASH key.


Thanks,
Khushboo



--
Akshay Joshi
Sr. Software Architect


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fixed dashboard reloading even if subnodes under ada
Next
From: Aditya Toshniwal
Date:
Subject: [pgAdmin4][RM2136] Include "?" on all routes that are loaded