Hi,
On 2022-11-25 11:41:45 +0000, Hayato Kuroda (Fujitsu) wrote:
> Sorry for my late reply. I understood that we got agreement the basic design of first version. Thanks!
> I attached new version patches.
This is failing on cfbot / CI, as have prior versions.
https://cirrus-ci.com/github/postgresql-cfbot/postgresql/commitfest%2F40%2F3388
https://api.cirrus-ci.com/v1/artifact/task/6655254493134848/testrun/build/testrun/postgres_fdw/regress/regression.diffs
diff -U3 /tmp/cirrus-ci-build/contrib/postgres_fdw/expected/postgres_fdw.out
/tmp/cirrus-ci-build/build/testrun/postgres_fdw/regress/results/postgres_fdw.out
--- /tmp/cirrus-ci-build/contrib/postgres_fdw/expected/postgres_fdw.out 2022-12-06 05:21:33.906116000 +0000
+++ /tmp/cirrus-ci-build/build/testrun/postgres_fdw/regress/results/postgres_fdw.out 2022-12-06 05:27:24.929694000
+0000
@@ -11732,10 +11732,9 @@
-- execute check function. This should return false on supported platforms,
-- otherwise return true.
SELECT postgres_fdw_verify_connection_states('loopback');
-WARNING: 08006
postgres_fdw_verify_connection_states
---------------------------------------
- f
+ t
(1 row)
ABORT;
The failure happens everywhere except on linux, so presumably there's some
portability issue in the patch.
I've set the patch as waiting on author for now.
Note that you can test CI in your own repository, as explained in src/tools/ci/README
Greetings,
Andres Freund