postgres.git / summary / log / commit / refs

commit    6f1cf2efbd0199c58db9f256ade48f42f56cab49
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Fri May 19 14:57:46 2023 +0000
First released in: 16.0, 11.21, 12.16, 13.12, 14.9, 15.4

    Avoid naming conflict between transactions.sql and namespace.sql.
    
    Commits 681d9e462 et al added a test case in namespace.sql that
    implicitly relied on there not being a table "public.abc".
    However, the concurrently-run transactions.sql test creates precisely
    such a table, so with the right timing you'd get a failure.
    Creating a table named as generically as "abc" in a common schema
    seems like bad practice, so fix this by changing the name of
    transactions.sql's table.  (Compare 2cf8c7aa4.)
    
    Marina Polyakova
    
    Discussion: https://postgr.es/m/80d0201636665d82185942e7112257b4@postgrespro.ru


 src/test/regress/expected/transactions.out | 18 +++++++++---------
 src/test/regress/sql/transactions.sql      | 18 +++++++++---------
 2 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/src/test/regress/expected/transactions.out b/src/test/regress/expected/transactions.out
index 69e176c5259..df704bc3f58 100644
--- a/src/test/regress/expected/transactions.out
+++ b/src/test/regress/expected/transactions.out
@@ -571,10 +571,10 @@ drop function inverse(int);
 -- performed in the aborted subtransaction
 begin;
 savepoint x;
-create table abc (a int);
-insert into abc values (5);
-insert into abc values (10);
-declare foo cursor for select * from abc;
+create table trans_abc (a int);
+insert into trans_abc values (5);
+insert into trans_abc values (10);
+declare foo cursor for select * from trans_abc;
 fetch from foo;
  a 
 ---
@@ -587,11 +587,11 @@ fetch from foo;
 ERROR:  cursor "foo" does not exist
 commit;
 begin;
-create table abc (a int);
-insert into abc values (5);
-insert into abc values (10);
-insert into abc values (15);
-declare foo cursor for select * from abc;
+create table trans_abc (a int);
+insert into trans_abc values (5);
+insert into trans_abc values (10);
+insert into trans_abc values (15);
+declare foo cursor for select * from trans_abc;
 fetch from foo;
  a 
 ---
diff --git a/src/test/regress/sql/transactions.sql b/src/test/regress/sql/transactions.sql
index 2e3739fd6c4..5ff290a46ea 100644
--- a/src/test/regress/sql/transactions.sql
+++ b/src/test/regress/sql/transactions.sql
@@ -358,10 +358,10 @@ drop function inverse(int);
 begin;
 
 savepoint x;
-create table abc (a int);
-insert into abc values (5);
-insert into abc values (10);
-declare foo cursor for select * from abc;
+create table trans_abc (a int);
+insert into trans_abc values (5);
+insert into trans_abc values (10);
+declare foo cursor for select * from trans_abc;
 fetch from foo;
 rollback to x;
 
@@ -371,11 +371,11 @@ commit;
 
 begin;
 
-create table abc (a int);
-insert into abc values (5);
-insert into abc values (10);
-insert into abc values (15);
-declare foo cursor for select * from abc;
+create table trans_abc (a int);
+insert into trans_abc values (5);
+insert into trans_abc values (10);
+insert into trans_abc values (15);
+declare foo cursor for select * from trans_abc;
 
 fetch from foo;
 


[parent: fc7dc728d1a6]