[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in Synonym node EPAS9.2

2017-06-05 Thread Murtuza Zabuawala
Hi, PFA patch to fix the issue in Synonym node for EPAS9.2+ where Packages are not displaying as a target objects. RM#1813 -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company diff --git

[pgadmin-hackers] Re: [pgAdmin4][Patch][Feature #1971]: Remember column sizes between executions of the same query in the query tool

2017-06-05 Thread Surinder Kumar
Staged changes are missed in previous patch, so please ignore. Please find attached updated patch. On Mon, Jun 5, 2017 at 4:29 PM, Surinder Kumar < surinder.ku...@enterprisedb.com> wrote: > Hi > > This patch contains two fixes: > > 1) In Query/tool or Edit grid, the width of table column header

[pgadmin-hackers] [pgAdmin4][Patch][Feature #1971]: Remember column sizes between executions of the same query in the query tool

2017-06-05 Thread Surinder Kumar
Hi This patch contains two fixes: 1) In Query/tool or Edit grid, the width of table column header is fixed depending on the column type(int, boolean, char etc.) due to which the column name or type appears cut from right and doesn't looks good from user point of view. The main concern was to

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-05 Thread Dave Page
On Fri, Jun 2, 2017 at 9:01 AM, Akshay Joshi wrote: > Hi All > > Following are the further implementation updates to support Declarative > Partitioning: > >- Show all the existing partitions of the parent table in Partitions >tab (Refer

[pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2381 - "Create Script" for view assigns incorrect schema to trigger functions in "public" schema.

2017-06-05 Thread Khushboo Vashi
Hi, Please find the attached patch to fix RM 2381: "Create Script" for view assigns incorrect schema to trigger functions in "public" schema. Fixed the trigger function schema name in the view SQL tab. Thanks, Khushboo diff --git

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Load module's JS files only when required

2017-06-05 Thread Surinder Kumar
Hi Dave, Please find attached patch for minifying CSS files and optimize images using Webpack. *Steps to run:* 1) After applying patch, go to web directory and run npm install on terminal to install packages which are used to parse/minify CSS files. 2) Run npm run build which creates dist

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-05 Thread Dave Page
Hi On Mon, Jun 5, 2017 at 4:34 PM, Shirley Wang wrote: > Hi! > > Here's a summary of the interviews thus far: > > DBAs tend to create a partition strategy based on their experience and > some alignment on their end users goals (analysts, report writers, and app > developers).

Re: [pgadmin-hackers] Style guide live website - looking for feedback

2017-06-05 Thread Dave Page
On Wed, May 31, 2017 at 7:39 PM, Shirley Wang wrote: > Hi all, > > We've started compiling some of the agreed upon styles into a site that is > accessible for all developers. > > Link to the site: http://pgadmin4-styleguide.cfapps.io/index.html > > As of now, we have colors,

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-05 Thread Shirley Wang
Hi! Here's a summary of the interviews thus far: DBAs tend to create a partition strategy based on their experience and some alignment on their end users goals (analysts, report writers, and app developers). Once that partition strategy is created, they are usually forced to iterate on that

[pgadmin-hackers] pgAdmin 4 commit: Handle procedure flags (IMMUTABLE STRICT SECURITY DEF

2017-06-05 Thread Dave Page
Handle procedure flags (IMMUTABLE STRICT SECURITY DEFINER PARALLEL RESTRICTED) properly in RE-SQL on EPAS. Fixes #2280 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=3f150f3f4ec395e62db3f51accb601539863b5f1 Author: Murtuza Zabuawala

[pgadmin-hackers] pgAdmin 4 commit: Cache statistics more reliably. Fixes #2357

2017-06-05 Thread Dave Page
Cache statistics more reliably. Fixes #2357 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=a4d86d24e6e664075d85461d816504c9d322f648 Author: Murtuza Zabuawala Modified Files --

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-05 Thread Robert Eckhardt
On Mon, Jun 5, 2017 at 11:45 AM, Dave Page wrote: > > > The former is what I was bleating about when I said we needed to expose > partitions to the user. The latter isn't relevant - declarative > partitioning in Postgres doesn't use inheritance. > The former is certainly the

Re: [pgadmin-hackers] [pgAdmin4][Patch] To display proper SQL for Procedure node

2017-06-05 Thread Dave Page
Thanks, patch applied. On Tue, May 30, 2017 at 10:13 AM, Murtuza Zabuawala wrote: > Hi, > > PFA patch to fix to display procedure options like IMMUTABLE STRICT SECURITY > DEFINER PARALLEL RESTRICTED in SQL (Reverse Engineered). > RM#2280 > > -- > Regards, >

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-05 Thread Robert Eckhardt
I guess what I didn't say is YES. What Akshay and Ashesh are building is going to absolutely be fundamental to any workflows being defined in these interviews. -- Rob On Mon, Jun 5, 2017 at 12:17 PM, Robert Eckhardt wrote: > On Mon, Jun 5, 2017 at 11:45 AM, Dave Page

[pgadmin-hackers] pgAdmin 4 commit: Display and allow toggling of trigger enable/disable

2017-06-05 Thread Dave Page
Display and allow toggling of trigger enable/disable status from the trigger dialogue. Fixes #2386 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=9e98ded06c2c80968522db1c731e941a5b8d5f32 Modified Files --

Re: [pgadmin-hackers] [pgAdmin4][PATCH] Improvements to Query Results Grid User Experience

2017-06-05 Thread Robert Eckhardt
On Mon, Jun 5, 2017 at 9:09 AM, Surinder Kumar < surinder.ku...@enterprisedb.com> wrote: > Hi Matthew, > > Couple of review comments: > 1) Clicking on a new added row(after save) results in console error - > screenshot attached. > We'll check this out as soon as we can. > 2) If i drag to

Re: [pgadmin-hackers] Re: [pgAdmin4][Patch][Feature #1971]: Remember column sizes between executions of the same query in the query tool

2017-06-05 Thread Shruti B Iyer
Hi Surinder! We reviewed this patch. The changes look good and we especially like that you have extracted out the new utility functions and the epicRandomString function too. This patch will likely affect the Query Results patch that is currently under review

[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in procedure debugging in EPAS

2017-06-05 Thread Murtuza Zabuawala
Hi, PFA patch to fix the issue where debugger module is not throwing proper error for unsupported argument mode type in procedure. RM#1466 @Neel, Can you please do quick review of this patch? -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab.

2017-06-05 Thread Khushboo Vashi
Hi, Fixed PostGIS datatype on SQL tab for Table, column, Foreign Table and Type node. Please find the attached updated patch. Thanks, Khushboo On Thu, Jun 1, 2017 at 6:36 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > > On Thu, Jun 1, 2017 at 5:27 PM, Khushboo Vashi < >

[pgadmin-hackers] [PATCH] extract function from sqleditor

2017-06-05 Thread George Gelashvili
Hello Hackers! Re. https://www.postgresql.org/message-id/CAFS4TJb- 7VTQQrnOi0g6MaoxMfEK9LzCds2yG%2BByS-mLHwydQA%40mail.gmail.com We extracted the function bound to onSelectedRangesChanged from sqleditor! - we read through the existing implementation and wrapped it in