Postgres Pro
Downloads
Home   >   mailing lists

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE
Date
Msg-id CA+OCxoytpKWGnpSdaUXa1_pHSyXUiN29DnU1x49rOx5Q7YN+mA@mail.gmail.com
Whole thread Raw
In response to [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Responses Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAINoption with IE
List pgadmin-hackers
Tree view
Akshay, can you review/commit this please?

On Fri, Apr 21, 2017 at 10:25 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

Please find the patch to fix the issue while using ANALYZE option in IE.

Issue:
We were using .startsWith() function to check the sql if it is already starts with explain clause or not, which was added in commit: c758920890309e50f47a9427039cf735ddf04d37

But IE do not support .startsWith() function.


FeatureChromeFirefox (Gecko)EdgeInternet ExplorerOperaSafari
Basic support4117 (17)(Yes)No support289


But IE do not support .startsWith() function.

I have also changed the code for File Manager module as well for the same issue.

So affected modules are,
- Query tool
- File Manager

Please review.
RM#2347


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


--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers




--
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: Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue withdisplaying bigint[] values in query tool
Next
From: Dave Page
Date:
Subject: Re: [pgadmin-hackers][patch] Dependents and Dependencies in GreenPlum