Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit b15a8c9

Browse files
committed
Clean up duplicate table and function names in regression tests.
Many of the objects we create during the regression tests are put in the public schema, so that using the same names in different regression tests creates a hazard of test failures if any two such scripts run concurrently. This patch cleans up a bunch of latent hazards of that sort, as well as two live hazards. The current situation in this regard is far worse than it was a year or two back, because practically all of the partitioning-related test cases have reused table names with enthusiasm. I despaired of cleaning up that mess within the five most-affected tests (create_table, alter_table, insert, update, inherit); fortunately those don't run concurrently. Other than partitioning problems, most of the issues boil down to using names like "foo", "bar", "tmp", etc, without thought for the fact that other test scripts might use similar names concurrently. I've made an effort to make all such names more specific. One of the live hazards was that commit 7421f4b caused with.sql to create a table named "test", conflicting with a similarly-named table in alter_table.sql; this was exposed in the buildfarm recently. The other one was that join.sql and transactions.sql both create tables named "foo" and "bar"; but join.sql's uses of those names date back only to December or so. Since commit 7421f4b was back-patched to v10, back-patch a minimal fix for that problem. The rest of this is just future-proofing. Discussion: https://postgr.es/m/4627.1521070268@sss.pgh.pa.us
1 parent 12bceca commit b15a8c9

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/test/regress/expected/with.out

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2280,7 +2280,7 @@ LINE 1: WITH test AS (SELECT 42) INSERT INTO test VALUES (1);
22802280
-- check response to attempt to modify table with same name as a CTE (perhaps
22812281
-- surprisingly it works, because CTEs don't hide tables from data-modifying
22822282
-- statements)
2283-
create table test (i int);
2283+
create temp table test (i int);
22842284
with test as (select 42) insert into test select * from test;
22852285
select * from test;
22862286
i

src/test/regress/sql/with.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1035,7 +1035,7 @@ WITH test AS (SELECT 42) INSERT INTO test VALUES (1);
10351035
-- check response to attempt to modify table with same name as a CTE (perhaps
10361036
-- surprisingly it works, because CTEs don't hide tables from data-modifying
10371037
-- statements)
1038-
create table test (i int);
1038+
create temp table test (i int);
10391039
with test as (select 42) insert into test select * from test;
10401040
select * from test;
10411041
drop table test;

0 commit comments

Comments
 (0)