Обсуждение: buildfarm failures, src/test/recovery

Hi Andrew,

It appears to me that 2ef6f11b0c77ec323c688ddfd98ffabddb72c11d broke src/test/recovery.

It looks like the following fixes it.  Care to review and push?  Or perhaps just revert that commit?

diff --git a/src/test/regress/expected/jsonb_sqljson.out b/src/test/regress/expected/jsonb_sqljson.out
index 230cfd3bfd..ee16f2770a 100644
--- a/src/test/regress/expected/jsonb_sqljson.out
+++ b/src/test/regress/expected/jsonb_sqljson.out
@@ -2088,7 +2088,7 @@ ERROR:  only string constants supported in JSON_TABLE path specification
 LINE 1: SELECT * FROM JSON_TABLE(jsonb '{"a": 123}', '$' || '.' || '...
                                                      ^
 -- Test parallel JSON_VALUE()
-CREATE UNLOGGED TABLE test_parallel_jsonb_value AS
+CREATE TABLE test_parallel_jsonb_value AS
 SELECT i::text::jsonb AS js
 FROM generate_series(1, 500000) i;
 -- Should be non-parallel due to subtransactions
diff --git a/src/test/regress/sql/jsonb_sqljson.sql b/src/test/regress/sql/jsonb_sqljson.sql
index 866c708a4d..2fc41a6df5 100644
--- a/src/test/regress/sql/jsonb_sqljson.sql
+++ b/src/test/regress/sql/jsonb_sqljson.sql
@@ -948,7 +948,7 @@ SELECT JSON_QUERY(jsonb '{"a": 123}', 'error' || ' ' || 'error');
 SELECT * FROM JSON_TABLE(jsonb '{"a": 123}', '$' || '.' || 'a' COLUMNS (foo int));

 -- Test parallel JSON_VALUE()
-CREATE UNLOGGED TABLE test_parallel_jsonb_value AS
+CREATE TABLE test_parallel_jsonb_value AS
 SELECT i::text::jsonb AS js
 FROM generate_series(1, 500000) i;

—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On 4/6/22 13:42, Mark Dilger wrote:
> Hi Andrew,
>
> It appears to me that 2ef6f11b0c77ec323c688ddfd98ffabddb72c11d broke src/test/recovery.
>
> It looks like the following fixes it.  Care to review and push?  Or perhaps just revert that commit?


Fixed here
https://git.postgresql.org/pg/commitdiff/14d3f24fa8a21f8a7e66f1fc60253a1e11410bf3


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com