Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Avoid rare race condition in privileges.sql regression test.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 16 Oct 2018 17:56:58 +0000 (13:56 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 16 Oct 2018 17:56:58 +0000 (13:56 -0400)
We created a temp table, then switched to a new session, leaving
the old session to clean up its temp objects in background.  If that
took long enough, the eventual attempt to drop the user that owns
the temp table could fail, as exhibited today by sidewinder.
Fix by dropping the temp table explicitly when we're done with it.

It's been like this for quite some time, so back-patch to all
supported branches.

Report: https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=sidewinder&dt=2018-10-16%2014%3A45%3A00

src/test/regress/expected/privileges.out
src/test/regress/sql/privileges.sql

index ec84ea0e259ac0ad2ed0370eb5ed7bdda37d226b..b7b8c813a037127364040c84adfa3cdca25381a1 100644 (file)
@@ -1155,6 +1155,7 @@ select has_column_privilege('mytable',2::int2,'select');
  
 (1 row)
 
+drop table mytable;
 -- Grant options
 SET SESSION AUTHORIZATION regress_user1;
 CREATE TABLE atest4 (a int);
index fee0c3b34b4f47bd6780038fb89a04c486738981..1ae079349fba32ffeb53d00e1054694558787ead 100644 (file)
@@ -715,6 +715,7 @@ select has_column_privilege('mytable','........pg.dropped.2........','select');
 select has_column_privilege('mytable',2::int2,'select');
 revoke select on table mytable from regress_user3;
 select has_column_privilege('mytable',2::int2,'select');
+drop table mytable;
 
 -- Grant options