Thread: [RM#4468] Add RE-SQL tests for Type node

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

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

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