Re: pgAdmin4: Cleanup in role dependency test cases - Mailing list pgadmin-hackers

From Dave Page
Subject Re: pgAdmin4: Cleanup in role dependency test cases
Date
Msg-id CA+OCxoztUK7hf9E4msuLgX9LqBxGacBDKbMRmLHPmsFfxBj5uQ@mail.gmail.com
Whole thread Raw
In response to pgAdmin4: Cleanup in role dependency test cases  (Navnath Gadakh <navnath.gadakh@enterprisedb.com>)
List pgadmin-hackers
Hi

On Wed, Aug 30, 2017 at 6:34 AM, Navnath Gadakh <navnath.gadakh@enterprisedb.com> wrote:
Hi Dave,

Role dependency test cases are failing due to database error 'ERROR: cache lookup failed for database XXXXXX'. Test-suite throwing error on "DROP ROLE <role_name>".
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 drop the shared dependencies correctly in role dependency test cases. 
PostgreSQL still thinks that Role owns 1 table in the database which is dropped now .Due to that role was not able to drop. 
This could be a bug in PG. Anyway, since the database is dropped, it is safe to manually remove the user as it is not of any use more.

It's never safe to remove a user like that - plus the user would need rolcatupdate privs for it to even work (which a superuser doesn't have by default).

Have you run the problem past the DB Server team?

Another point to note is that support for 9.2 will end very soon, so it might not be worth worrying about.

--
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: Teng Zhang
Date:
Subject: [pgAdmin4][Patch] ACI tree compatible with Greenplum
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch]: Allow user to provide custom SSL certificatesand provide .pgpass file