Postgres Pro
Downloads
Home   >   mailing lists

Re: [RM#4468] Add RE-SQL tests for Type node - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [RM#4468] Add RE-SQL tests for Type node
Date
Msg-id CA+OCxoxEa7HsDdiCbhF=+apjyC+xfBngh3Sch2f_7Huwv8OWuQ@mail.gmail.com
Whole thread Raw
In response to Re: [RM#4468] Add RE-SQL tests for Type node  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers
Tree view
Thanks, applied.

On Mon, Jul 15, 2019 at 1:16 PM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Dave,

PFA patch to fix the ACL order so that it RE-SQL tests for type node works consistently on all the platforms.

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



On Mon, Jul 15, 2019 at 2:55 PM Dave Page <dpage@pgadmin.org> wrote:
Thanks, patch applied.

On Mon, Jul 15, 2019 at 7:57 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

Please find patch for RE-SQL type node.

- Fixed a bug in update.sql which is found during writing tests
- Added logging for JSON file when it fails to load


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Stabilise ordering of ACLs in Type RE-SQL tests to pr
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fix sequence reverse engineered SQL generation with q