Re: pgAdmin 4 commit: Fix the feature tests failuers.

2017-08-29 Thread Harshal Dhumal
Hi Ashesh, I ran feature test case on mac couple times however I didn't get failure which Dave and you got. In my case feature test cases mostly falling in xxs_debugger test case. This was because as when we configure database sever to debug functions it adds 21 its own functions in database and

Re: pgAdmin 4 commit: Fix the feature tests failuers.

2017-08-29 Thread Dave Page
On Tue, Aug 29, 2017 at 8:41 AM, Harshal Dhumal < harshal.dhu...@enterprisedb.com> wrote: > Hi Ashesh, > > I ran feature test case on mac couple times however I didn't get failure > which Dave and you got. > In my case feature test cases mostly falling in xxs_debugger test case. > > This was becau

Re: pgAdmin 4 commit: Update docs to more accurately reflect the SERVER_MOD

2017-08-29 Thread Dave Page
Hi On Mon, Aug 28, 2017 at 7:51 AM, Akshay Joshi wrote: > Hi Dave > > Changes looks good to me, only one thing is that I also override > 'DATA_DIR' and 'TEST_SQLITE_PATH' parameter in my config_local.py, so we > should include both the parameters in 'desktop_deployment.rst' file. > DATA_DIR is

Re: [pgAdmin4][Patch]: RM_2596 - Query tool not working in Desktop Runtime on Mac OS X

2017-08-29 Thread Surinder Kumar
Hi Ashesh, Can you please review this patch please? Thanks, Surinder On Wed, Aug 16, 2017 at 3:43 PM, Surinder Kumar < surinder.ku...@enterprisedb.com> wrote: > Hi, > > Updated patch contains changes: > >- Enable definePlugin for development environment as well. Just adding >definePlugi

pgAdmin 4 commit: Cleanup feature tests. Fixes #2586

2017-08-29 Thread Dave Page
Cleanup feature tests. Fixes #2586 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=ceb9438000cea77f7054693a292cf4bd5850b4a0 Author: Harshal Dhumal Modified Files -- .../connect_to_server_feature_test.py | 84 - ..

Re: [pgAdmin4][RM2586] Cleanup feature test

2017-08-29 Thread Dave Page
Thanks, patch applied (test time now down to 209s :-) ). On Mon, Aug 28, 2017 at 11:04 AM, Harshal Dhumal < harshal.dhu...@enterprisedb.com> wrote: > Hi, > > Please find attached updated patch > > On Fri, Aug 25, 2017 at 2:32 PM, Dave Page wrote: > >> Hi >> >> On Thu, Aug 24, 2017 at 11:51 AM, H

pgAdmin 4 commit: Allow pgAdmin to run with config database versions fr

2017-08-29 Thread Dave Page
Allow pgAdmin to run with config database versions from the future. Fixes #2664 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=7835da267b390ce8fc97c68ca735e0e0c8bde94a Author: Surinder Kumar Modified Files -- web/pgadmin/__init__

Re: [pgAdmin4][Patch]: Ensure any changes to the config database are backward compatible

2017-08-29 Thread Dave Page
Thanks, patch applied. On Fri, Aug 25, 2017 at 3:12 PM, Surinder Kumar < surinder.ku...@enterprisedb.com> wrote: > Created an RM to track this > issue. > > On Fri, Aug 25, 2017 at 7:19 PM, Surinder Kumar < > surinder.ku...@enterprisedb.com> wrote: > >>

Re: [pgAdmin4][Patch]: Allow user to provide custom SSL certificates and provide .pgpass file

2017-08-29 Thread Dave Page
Hi On Fri, Aug 25, 2017 at 2:45 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi Dave, > > Please find updated patch, > - For displaying hidden files I have added preference option in Storage > section. > How painful would it be to include it on the file dialogue as well?

Re: [pgaweb][patch] Use https link for git repo

2017-08-29 Thread Matthew Kleiman
bump. On Tue, Aug 15, 2017 at 3:58 AM Matthew Kleiman wrote: > Hi Hackers, > > As per discussion on > https://www.postgresql.org/message-id/flat/CAGRPzo-_dSYr5Y1D%2B0uBLo39Bb6MVTg3%3DCM%3DhrmX7LdF%3DfQbhQ%40mail.gmail.com#CAGRPzo-_dSYr5Y1D+0uBLo39Bb6MVTg3=CM=hrmX7LdF=fq...@mail.gmail.com, > I've

Re: [pgaweb][patch] Use https link for git repo

2017-08-29 Thread Dave Page
On Tue, Aug 29, 2017 at 3:30 PM, Matthew Kleiman wrote: > bump. > > On Tue, Aug 15, 2017 at 3:58 AM Matthew Kleiman > wrote: > >> Hi Hackers, >> >> As per discussion on https://www.postgresql.org/message-id/flat/CAGRPzo-_ >> dSYr5Y1D%2B0uBLo39Bb6MVTg3%3DCM%3DhrmX7LdF%3DfQbhQ% >> 40mail.gmail.com

Re: [pgaweb][patch] Use https link for git repo

2017-08-29 Thread Magnus Hagander
On Tue, Aug 29, 2017 at 4:45 PM, Dave Page wrote: > > > On Tue, Aug 29, 2017 at 3:30 PM, Matthew Kleiman > wrote: > >> bump. >> >> On Tue, Aug 15, 2017 at 3:58 AM Matthew Kleiman >> wrote: >> >>> Hi Hackers, >>> >>> As per discussion on https://www.postgresql.org/ >>> message-id/flat/CAGRPzo-_d

Re: [pgaweb][patch] Use https link for git repo

2017-08-29 Thread Dave Page
On Tue, Aug 29, 2017 at 3:55 PM, Magnus Hagander wrote: > > > On Tue, Aug 29, 2017 at 4:45 PM, Dave Page wrote: > >> >> >> On Tue, Aug 29, 2017 at 3:30 PM, Matthew Kleiman >> wrote: >> >>> bump. >>> >>> On Tue, Aug 15, 2017 at 3:58 AM Matthew Kleiman >>> wrote: >>> Hi Hackers, A

pgAdmin4: Cleanup in role dependency test cases

2017-08-29 Thread Navnath Gadakh
Hi Dave, Role dependency test cases are failing due to database error *'ERROR: cache lookup failed for database XX'. *Test-suite throwing error on* "DROP ROLE ".* This is intermittent issue I was executing test cases on all servers (pg9.2-10 and ppas9.2-10) Somehow the 'DROP database' didn't

[pgAdmin4][Patch] ACI tree compatible with Greenplum

2017-08-29 Thread Teng Zhang
Hi Hackers, Attached is a patch that makes ACI tree works better with Greenplum. When connected to a Greenplum database server - Hide items that not work with GPDB, such as Triggers, FDW, FTS_*, etc - Tables/Views/Catalogs/Language node can expand correctly - Views/Languages/Catalogs can show pro