Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in handling of timestamp type

[Dave Page]

Hi

On Tue, Jan 31, 2017 at 5:19 AM, Murtuza Zabuawala
 wrote:
> Hi Dave,
>
> PFA updated patch.

This seems to display "timestamp(0) with[out] timezone" columns
correctly in both the properties panel and dialog now, but the size is
still ignored if I try to add a new column through the table or column
dialogue.

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

EnterpriseDB UK: 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

[pgadmin-hackers] pgAdmin 4 commit: Rename the 9.1_plus templates to "default" to more ap

[Dave Page]

Rename the 9.1_plus templates to "default" to more appropriately support 
Greenplum which is based on 8.3.

Branch
--
master

Details
---
http://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=6d05302ca106e1f07e1372284e00507dc3058fe2
Author: George Gelashvili 

Modified Files
--
.../servers/databases/casts/__init__.py|   2 +-
.../casts/templates/cast/sql/9.1_plus/create.sql   |  15 --
.../casts/templates/cast/sql/9.1_plus/delete.sql   |  14 --
.../templates/cast/sql/9.1_plus/functions.sql  |  17 --
.../cast/sql/9.1_plus/getsrcandtrgttype.sql|  46 -
.../casts/templates/cast/sql/9.1_plus/nodes.sql|  24 ---
.../templates/cast/sql/9.1_plus/properties.sql |  59 --
.../casts/templates/cast/sql/9.1_plus/sql.sql  |  44 -
.../casts/templates/cast/sql/9.1_plus/update.sql   |   6 -
.../casts/templates/cast/sql/default/create.sql|  15 ++
.../casts/templates/cast/sql/default/delete.sql|  14 ++
.../casts/templates/cast/sql/default/functions.sql |  17 ++
.../cast/sql/default/getsrcandtrgttype.sql |  46 +
.../casts/templates/cast/sql/default/nodes.sql |  24 +++
.../templates/cast/sql/default/properties.sql  |  59 ++
.../casts/templates/cast/sql/default/sql.sql   |  44 +
.../casts/templates/cast/sql/default/update.sql|   6 +
.../templates/foreign_servers/sql/9.1_plus/acl.sql |  27 ---
.../foreign_servers/sql/9.1_plus/create.sql|  40 
.../foreign_servers/sql/9.1_plus/delete.sql|   9 -
.../foreign_servers/sql/9.1_plus/dependents.sql|  12 --
.../foreign_servers/sql/9.1_plus/properties.sql|  27 ---
.../foreign_servers/sql/9.1_plus/update.sql|  87 -
.../templates/foreign_servers/sql/default/acl.sql  |  27 +++
.../foreign_servers/sql/default/create.sql |  40 
.../foreign_servers/sql/default/delete.sql |   9 +
.../foreign_servers/sql/default/dependents.sql |  12 ++
.../foreign_servers/sql/default/properties.sql |  27 +++
.../foreign_servers/sql/default/update.sql |  87 +
.../foreign_servers/user_mapping/__init__.py   |   2 +-
.../user_mappings/sql/9.1_plus/create.sql  |  13 --
.../user_mappings/sql/9.1_plus/delete.sql  |   9 -
.../user_mappings/sql/9.1_plus/properties.sql  |  23 ---
.../user_mappings/sql/9.1_plus/update.sql  |  37 
.../templates/user_mappings/sql/default/create.sql |  13 ++
.../templates/user_mappings/sql/default/delete.sql |   9 +
.../user_mappings/sql/default/properties.sql   |  23 +++
.../templates/user_mappings/sql/default/update.sql |  37 
.../foreign_data_wrappers/sql/9.1_plus/acl.sql |  27 ---
.../foreign_data_wrappers/sql/9.1_plus/create.sql  |  38 
.../foreign_data_wrappers/sql/9.1_plus/delete.sql  |   8 -
.../sql/9.1_plus/handlers.sql  |   5 -
.../sql/9.1_plus/properties.sql|  18 --
.../foreign_data_wrappers/sql/9.1_plus/update.sql  |  99 --
.../sql/9.1_plus/validators.sql|   5 -
.../foreign_data_wrappers/sql/default/acl.sql  |  27 +++
.../foreign_data_wrappers/sql/default/create.sql   |  38 
.../foreign_data_wrappers/sql/default/delete.sql   |   8 +
.../foreign_data_wrappers/sql/default/handlers.sql |   5 +
.../sql/default/properties.sql |  18 ++
.../foreign_data_wrappers/sql/default/update.sql   |  99 ++
.../sql/default/validators.sql |   5 +
.../templates/languages/sql/9.1_plus/acl.sql   |  23 ---
.../templates/languages/sql/9.1_plus/create.sql|  39 
.../templates/languages/sql/9.1_plus/delete.sql|   8 -
.../templates/languages/sql/9.1_plus/functions.sql |  15 --
.../languages/sql/9.1_plus/properties.sql  |  23 ---
.../templates/languages/sql/9.1_plus/sqlpane.sql   |  42 -
.../templates/languages/sql/9.1_plus/templates.sql |   7 -
.../templates/languages/sql/9.1_plus/update.sql|  62 ---
.../templates/languages/sql/default/acl.sql|  23 +++
.../templates/languages/sql/default/create.sql |  39 
.../templates/languages/sql/default/delete.sql |   8 +
.../templates/languages/sql/default/functions.sql  |  15 ++
.../templates/languages/sql/default/properties.sql |  23 +++
.../templates/languages/sql/default/sqlpane.sql|  42 +
.../templates/languages/sql/default/templates.sql  |   7 +
.../templates/languages/sql/default/update.sql |  62 +++
.../databases/schemas/catalog_objects/__init__.py  |   5 +-
.../schemas/catalog_objects/columns/__init__.py|   2 +-
.../catalog_object_column/sql/9.1_plus/depend.sql  |  10 -
.../catalog_object_column/sql/9.1_plus/nodes.sql   |   7 -
.../sql/9.1_plus/properties.sql|  41 -
.../catalog_object_column/sql/default/depend.sql   |  10 +
.../catalog_object_column/sql/default/nodes.sql|   7 +
.../sql/default/properties.sql |  41 +
.../catalog_object/sql/pg/9.1_plus/nodes.sql 

Re: [pgadmin-hackers][Patch] Show databases for Greenplum

[Dave Page]

Thanks George, patch applied.

Fahar; whilst this is a technically simple patch, it was also very
large. Please keep an eye out for any issues.

On Tue, Jan 31, 2017 at 10:08 PM, George Gelashvili
 wrote:
> Hi Hackers,
>
> We're working on https://redmine.postgresql.org/issues/2132
> This patch allows pgAdmin to list databases and schemas when connected to a
> Greenplum server.
>
> The fix was largely allowing pgadmin to load templates that work with
> Greenplum. To achieve this, we used the versioned template loader in more
> places where "9.1_plus" was previously hardcoded.
> We also created a set of "default" template directories so that Greenplum
> will load the templates that were tagged as 9.1 for now.
>
> Tira & George
>
>
> --
> 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


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

Re: [pgadmin-hackers][Patch] Show databases for Greenplum

[Fahar Abbas]

I was also thinking with same pattern and we need to verify this change
thoroughly.

Thanks Dave.

Regards,

On Wed, Feb 1, 2017 at 2:59 PM, Dave Page  wrote:

> Thanks George, patch applied.
>
> Fahar; whilst this is a technically simple patch, it was also very
> large. Please keep an eye out for any issues.
>
> On Tue, Jan 31, 2017 at 10:08 PM, George Gelashvili
>  wrote:
> > Hi Hackers,
> >
> > We're working on https://redmine.postgresql.org/issues/2132
> > This patch allows pgAdmin to list databases and schemas when connected
> to a
> > Greenplum server.
> >
> > The fix was largely allowing pgadmin to load templates that work with
> > Greenplum. To achieve this, we used the versioned template loader in more
> > places where "9.1_plus" was previously hardcoded.
> > We also created a set of "default" template directories so that Greenplum
> > will load the templates that were tagged as 9.1 for now.
> >
> > Tira & George
> >
> >
> > --
> > 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
>



-- 
Syed Fahar Abbas
Quality Management Group

EnterpriseDB Corporation
Phone Office: +92-51-835-8874
Phone Direct: +92-51-8466803
Mobile: +92-333-5409707
Skype ID: syed.fahar.abbas
Website: www.enterprisedb.com

Re: [pgadmin-hackers] PATCH: To fix the issue in Database node (pgAdmin4)

[Murtuza Zabuawala]

Hi Dave,

PFA updated patch, issue was conn.release(did=did) was removing all
connection for same database from connection manager.
We have now created new connection for offline updates using "conn_id"
parameter instead of did.

Please review.

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

On Mon, Jan 30, 2017 at 8:03 PM, Dave Page  wrote:

> Hi
>
> On Mon, Jan 30, 2017 at 5:31 AM, Murtuza Zabuawala
>  wrote:
> > Hi Dave,
> >
> > Please find updated patch, Tested against maintenance database as well.
>
> I can change parameters on the maintenance DB now, but it still
> crashes if I try to change the comment. Both operations on other
> databases seem to work OK.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>


RM_2024_v5.patch
Description: Binary data

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

[pgadmin-hackers] pgAdmin 4 commit: Properly handle setting comments and other options on

[Dave Page]

Properly handle setting comments and other options on databases with allowconn 
= False. Fixes #2024

Branch
--
master

Details
---
http://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=2d442da9ffd7acbc97fc28e8dc95e043fefa1617
Author: Murtuza Zabuawala 

Modified Files
--
.../server_groups/servers/databases/__init__.py| 173 -
web/pgadmin/utils/driver/psycopg2/__init__.py  |   2 +
2 files changed, 100 insertions(+), 75 deletions(-)


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

Re: [pgadmin-hackers] PATCH: To fix the issue in Database node (pgAdmin4)

[Dave Page]

Thanks - patch applied!

On Wed, Feb 1, 2017 at 10:08 AM, Murtuza Zabuawala
 wrote:
> Hi Dave,
>
> PFA updated patch, issue was conn.release(did=did) was removing all
> connection for same database from connection manager.
> We have now created new connection for offline updates using "conn_id"
> parameter instead of did.
>
> Please review.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On Mon, Jan 30, 2017 at 8:03 PM, Dave Page  wrote:
>>
>> Hi
>>
>> On Mon, Jan 30, 2017 at 5:31 AM, Murtuza Zabuawala
>>  wrote:
>> > Hi Dave,
>> >
>> > Please find updated patch, Tested against maintenance database as well.
>>
>> I can change parameters on the maintenance DB now, but it still
>> crashes if I try to change the comment. Both operations on other
>> databases seem to work OK.
>>
>> --
>> 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


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

[pgadmin-hackers] [pgAdmin4][Patch] Displaying Zero in Slickgrid editor

[Murtuza Zabuawala]

Hi,

PFA minor patch to display value '0'  in Read only text editor.
RM#2138


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
diff --git a/web/pgadmin/static/js/slickgrid/slick.pgadmin.editors.js 
b/web/pgadmin/static/js/slickgrid/slick.pgadmin.editors.js
index 19ad11d..3ef4f5a 100644
--- a/web/pgadmin/static/js/slickgrid/slick.pgadmin.editors.js
+++ b/web/pgadmin/static/js/slickgrid/slick.pgadmin.editors.js
@@ -550,7 +550,16 @@
 };
 
 this.loadValue = function (item) {
-  defaultValue = item[args.column.field] || "";
+  var value = item[args.column.field];
+  // Check if value is null or undefined
+  if(
+(value === undefined && typeof value === "undefined")
+||
+(value === undefined && typeof value === "undefined")
+  ) {
+value = ""
+  }
+  defaultValue = value;
   $input.val(defaultValue);
   $input[0].defaultValue = defaultValue;
   $input.select();

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

Re: [pgadmin-hackers] [pgAdmin4][Patch] Displaying Zero in Slickgrid editor

[Murtuza Zabuawala]

Thanks Dave.

Tested and working fine.

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

On Wed, Feb 1, 2017 at 7:20 PM, Dave Page  wrote:

> Hi
>
> On Wed, Feb 1, 2017 at 12:18 PM, Murtuza Zabuawala
>  wrote:
> > Hi,
> >
> > PFA minor patch to display value '0'  in Read only text editor.
> > RM#2138
>
> Thanks - I committed a modified version of the patch; yours had the
> same tests duplicated and joined with an or conditional.
>
> Please double check it's OK - it looks fine to me and seems to work as
> expected with null, zero and one.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Re: [pgadmin-hackers] [pgAdmin4][Patch] Displaying Zero in Slickgrid editor

[Dave Page]

Hi

On Wed, Feb 1, 2017 at 12:18 PM, Murtuza Zabuawala
 wrote:
> Hi,
>
> PFA minor patch to display value '0'  in Read only text editor.
> RM#2138

Thanks - I committed a modified version of the patch; yours had the
same tests duplicated and joined with an or conditional.

Please double check it's OK - it looks fine to me and seems to work as
expected with null, zero and one.

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

EnterpriseDB UK: 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

[pgadmin-hackers] pgAdmin 4 commit: Fix display of zeros in read-only grid editors. Fixes

[Dave Page]

Fix display of zeros in read-only grid editors. Fixes #2138

Branch
--
master

Details
---
http://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=687a793383fa2531c8c2a6a9f82213c5779e470a
Author: Murtuza Zabuawala 

Modified Files
--
web/pgadmin/static/js/slickgrid/slick.pgadmin.editors.js | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)


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

Re: [pgadmin-hackers] Re: PATCH: RM# 1679 - Background process for "restore" not reporting status back to pgAdmin

[Ashesh Vashi]

Hi Fahar,

Can you please test the updated patch?
I have fixed one issue on windows, and also added logging information in
the background process execution for future issue analysis.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



*http://www.linkedin.com/in/asheshvashi*


On Mon, Jan 23, 2017 at 12:19 PM, Fahar Abbas 
wrote:

> Hi Murtaza,
>
> I am using Administrator user on 2012 server . Can you please check this
> behavior on my machine through skype or google talk?
>
> Regards,
>
>
> On Mon, Jan 23, 2017 at 10:07 AM, Murtuza Zabuawala  enterprisedb.com> wrote:
>
>> I tested with Python-2.7 on Windows10 (32-bit) & on MacOS I tested with
>> Python3.5 (64-bit), It is working as expected,
>>
>> I suspect that as you are running with Server 2012, permissions might be
>> the issue, Can you try running with Administrator?
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> On Sat, Jan 21, 2017 at 1:18 AM, Fahar Abbas <
>> fahar.ab...@enterprisedb.com> wrote:
>>
>>> I tested in Google chrome Windows 2012 R2 64 with Python 2.7.11. It
>>> might be possible that you tested on Python 3+.
>>>
>>> Please confirm?
>>>
>>> On Fri, Jan 20, 2017 at 6:49 PM, Murtuza Zabuawala <
>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>
 That's strange.

 I tested in Chrome, Need check in IE.

 Meanwhile can try with Chrome/Firefox?

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

 On Fri, Jan 20, 2017 at 7:01 PM, Fahar Abbas <
 fahar.ab...@enterprisedb.com> wrote:

> Hi Murtaza,
>
> I just applied this patch and issue is still not resolved.
>
> In case of Maintenance DB and Backup database message, *Maintenance
> job created* and *Backup job created*  displayed but no other
> dialogue box displayed either process is completed or not.
>
> Screen-shot and pgadmin.log file is attached.
>
> Kind Regards,
>
>
> On Fri, Jan 20, 2017 at 5:35 PM, Murtuza Zabuawala <
> murtuza.zabuaw...@enterprisedb.com> wrote:
>
>> Hi Fahar,
>>
>> Could you please test this updated patch?
>>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> On Tue, Jan 17, 2017 at 3:45 PM, Fahar Abbas <
>> fahar.ab...@enterprisedb.com> wrote:
>>
>>> Thanks Dave.
>>>
>>> On Tue, Jan 17, 2017 at 3:07 PM, Dave Page 
>>> wrote:
>>>
 Not surprising. Don't bother testing that any more for now thanks.
 I suspect it's a simple bug that Ashesh can fix pronto.

 On Tue, Jan 17, 2017 at 10:02 AM, Fahar Abbas <
 fahar.ab...@enterprisedb.com> wrote:

> Issue is also reproducible with PG-9.6 server.
>
> Regards,
>
>
> On Tue, Jan 17, 2017 at 2:55 PM, Fahar Abbas <
> fahar.ab...@enterprisedb.com> wrote:
>
>> Please attached the log files output from the terminal and
>> pgAdmin4.log.
>>
>> Kind Regards,
>>
>>
>> On Tue, Jan 17, 2017 at 2:46 PM, Dave Page 
>> wrote:
>>
>>> Thanks. What log output do you get?
>>>
>>> On Tue, Jan 17, 2017 at 9:44 AM, Fahar Abbas <
>>> fahar.ab...@enterprisedb.com> wrote:
>>>
 Hi Dave.

 After applying this patch on Windows 2012 server 64 with Python
 2.7.11 on EPAS-9.6 server, Maintenance DB and Backup database 
 failed.

 Please find attached the snapshot.

 Kind Regards,

 On Wed, Jan 11, 2017 at 6:19 PM, Dave Page 
 wrote:

> Hi
>
> On Wed, Jan 11, 2017 at 12:28 PM, Ashesh Vashi <
> ashesh.va...@enterprisedb.com> wrote:
>
>> Hi Dave,
>>
>> On Wed, Jan 11, 2017 at 12:03 PM, Dave Page <
>> dp...@pgadmin.org> wrote:
>>
>>>
>>> KeyError: u'end_time'
>>>
>>> The status file contains:
>>>
>>> {"start_time": "2017-01-11 06:27:20.939703 +", "pid":
>>> 49363, "exit_code": 0, "end_time": "2017-01-11 06:27:28.613456 
>>> +"}
>>>
>>
>> Thanks for reviewing the patch.
>> Please find the updated patch.
>>
>> Changes:
>> - Using now ValueError instead of json.JSONDecodeError, which
>> is subclass of ValueError, to allow to it to work with 2.6+ 

Re: [pgadmin-hackers] Re: PATCH: RM# 1679 - Background process for "restore" not reporting status back to pgAdmin

[Fahar Abbas]

Sure Ashesh,

I will test and update the testing status accordingly.

On Thu, Feb 2, 2017 at 11:17 AM, Ashesh Vashi  wrote:

> Hi Fahar,
>
> Can you please test the updated patch?
> I have fixed one issue on windows, and also added logging information in
> the background process execution for future issue analysis.
>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company
> 
>
>
> *http://www.linkedin.com/in/asheshvashi*
> 
>
> On Mon, Jan 23, 2017 at 12:19 PM, Fahar Abbas <
> fahar.ab...@enterprisedb.com> wrote:
>
>> Hi Murtaza,
>>
>> I am using Administrator user on 2012 server . Can you please check this
>> behavior on my machine through skype or google talk?
>>
>> Regards,
>>
>>
>> On Mon, Jan 23, 2017 at 10:07 AM, Murtuza Zabuawala <
>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>
>>> I tested with Python-2.7 on Windows10 (32-bit) & on MacOS I tested with
>>> Python3.5 (64-bit), It is working as expected,
>>>
>>> I suspect that as you are running with Server 2012, permissions might be
>>> the issue, Can you try running with Administrator?
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Sat, Jan 21, 2017 at 1:18 AM, Fahar Abbas <
>>> fahar.ab...@enterprisedb.com> wrote:
>>>
 I tested in Google chrome Windows 2012 R2 64 with Python 2.7.11. It
 might be possible that you tested on Python 3+.

 Please confirm?

 On Fri, Jan 20, 2017 at 6:49 PM, Murtuza Zabuawala <
 murtuza.zabuaw...@enterprisedb.com> wrote:

> That's strange.
>
> I tested in Chrome, Need check in IE.
>
> Meanwhile can try with Chrome/Firefox?
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On Fri, Jan 20, 2017 at 7:01 PM, Fahar Abbas <
> fahar.ab...@enterprisedb.com> wrote:
>
>> Hi Murtaza,
>>
>> I just applied this patch and issue is still not resolved.
>>
>> In case of Maintenance DB and Backup database message, *Maintenance
>> job created* and *Backup job created*  displayed but no other
>> dialogue box displayed either process is completed or not.
>>
>> Screen-shot and pgadmin.log file is attached.
>>
>> Kind Regards,
>>
>>
>> On Fri, Jan 20, 2017 at 5:35 PM, Murtuza Zabuawala <
>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>
>>> Hi Fahar,
>>>
>>> Could you please test this updated patch?
>>>
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Tue, Jan 17, 2017 at 3:45 PM, Fahar Abbas <
>>> fahar.ab...@enterprisedb.com> wrote:
>>>
 Thanks Dave.

 On Tue, Jan 17, 2017 at 3:07 PM, Dave Page 
 wrote:

> Not surprising. Don't bother testing that any more for now thanks.
> I suspect it's a simple bug that Ashesh can fix pronto.
>
> On Tue, Jan 17, 2017 at 10:02 AM, Fahar Abbas <
> fahar.ab...@enterprisedb.com> wrote:
>
>> Issue is also reproducible with PG-9.6 server.
>>
>> Regards,
>>
>>
>> On Tue, Jan 17, 2017 at 2:55 PM, Fahar Abbas <
>> fahar.ab...@enterprisedb.com> wrote:
>>
>>> Please attached the log files output from the terminal and
>>> pgAdmin4.log.
>>>
>>> Kind Regards,
>>>
>>>
>>> On Tue, Jan 17, 2017 at 2:46 PM, Dave Page 
>>> wrote:
>>>
 Thanks. What log output do you get?

 On Tue, Jan 17, 2017 at 9:44 AM, Fahar Abbas <
 fahar.ab...@enterprisedb.com> wrote:

> Hi Dave.
>
> After applying this patch on Windows 2012 server 64 with
> Python 2.7.11 on EPAS-9.6 server, Maintenance DB and Backup 
> database failed.
>
> Please find attached the snapshot.
>
> Kind Regards,
>
> On Wed, Jan 11, 2017 at 6:19 PM, Dave Page 
> wrote:
>
>> Hi
>>
>> On Wed, Jan 11, 2017 at 12:28 PM, Ashesh Vashi <
>> ashesh.va...@enterprisedb.com> wrote:
>>
>>> Hi Dave,
>>>
>>> On Wed, Jan 11, 2017 at 12:03 PM, Dave Page <
>>> dp...@pgadmin.org> wrote:
>>>

 KeyError: u'end_time'

 The status file contains:

 {"start_time": "2017-01-11 06:27:20.939703 +", "pid":
 49363, "exit_code": 0, "end_time": "2017-01-11 06:27:28.613456