Re: [pgAdmin][RM-5741]: [Schema Diff] Revisit all the CREATE and DROP DDL's to add appropriate 'IF EXISTS', 'CASCADE' and 'CREATE OR REPLACE'

Hi Nikhil

As per the discussion, we need to create a flag for whether to add the 'IF EXISTS" clause or not. Please send the combined patch after fixing that for all the nodes.

On Mon, Sep 20, 2021 at 3:58 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:

Hi Akshay,

Please find the patch for RM-5741. 

Updated missing sql checks for following nodes:

1. Role

2. Resources Group

3. Schema

4. Language

5. Tablespace

Regards,

Nikhil Mohite

On Fri, Jul 30, 2021 at 1:23 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:

Thanks, the patch applied to the React_Porting branch as it is a big change. 

With the testing of each node after porting this patch also gets tested.

On Thu, Jul 29, 2021 at 8:34 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:

Hi Hackers,

Please find the attached patch for RM-5741: [Schema Diff] Revisit all the CREATE and DROP DDL's to add appropriate 'IF EXISTS', 'CASCADE' and 'CREATE OR REPLACE'

Tested RESQL and API test cases on PG-9.6 to 13 and EPAS-9.6 to 13.3.

--

Thanks & Regards,

Nikhil Mohite

Software Engineer.

EDB Postgres

Mob.No: +91-7798364578.

--

Thanks & Regards

Akshay Joshi

pgAdmin Hacker | Principal Software Architect

EDB Postgres

Mobile: +91 976-788-8246

--

Thanks & Regards

Akshay Joshi

pgAdmin Hacker | Principal Software Architect

EDB Postgres

Mobile: +91 976-788-8246