From: | Marina Polyakova <m(dot)polyakova(at)postgrespro(dot)ru> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Noah Misch <noah(at)leadboat(dot)com>, exclusion(at)gmail(dot)com |
Subject: | Re: Conflict between regression tests namespace & transactions due to recent changes |
Date: | 2023-05-15 20:23:18 |
Message-ID: | f62cd65e1b179783521f424a42ef3b27@postgrespro.ru |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2023-05-15 19:16, Tom Lane wrote:
> Marina Polyakova <m(dot)polyakova(at)postgrespro(dot)ru> writes:
>> IIUC the conflict was caused by
>
>> +SET search_path to public, test_ns_schema_1;
>> +CREATE SCHEMA test_ns_schema_2
>> + CREATE VIEW abc_view AS SELECT a FROM abc;
>
>> because the parallel regression test transactions had already created
>> the table abc and was trying to drop it.
>
> Hmm. I'd actually fix the blame on transactions.sql here. Creating
> a table named as generically as "abc" is horribly bad practice in
> a set of concurrent tests. namespace.sql is arguably okay, since
> it's creating that table name in a private schema.
>
> I'd be inclined to fix this by doing s/abc/something-else/g in
> transactions.sql.
>
> regards, tom lane
Maybe use a separate schema for all new objects in the transaction
test?.. See diff_set_tx_schema.patch.
--
Marina Polyakova
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
Attachment | Content-Type | Size |
---|---|---|
diff_set_tx_schema.patch | text/x-diff | 1.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Davis | 2023-05-15 21:03:23 | Re: Order changes in PG16 since ICU introduction |
Previous Message | Andres Freund | 2023-05-15 20:13:26 | Re: issue with meson builds on msys2 |