Re: [Proposal] Add foreign-server health checks infrastructure - Mailing list pgsql-hackers

From Fujii Masao
Subject Re: [Proposal] Add foreign-server health checks infrastructure
Date
Msg-id fd8f6186-9e1e-4b9a-92c5-e71e3697d381@oss.nttdata.com
Whole thread Raw
In response to Re: [Proposal] Add foreign-server health checks infrastructure  (Fujii Masao <masao.fujii@oss.nttdata.com>)
Responses RE: [Proposal] Add foreign-server health checks infrastructure
List pgsql-hackers

On 2024/07/26 22:44, Fujii Masao wrote:
> 
> 
> On 2024/07/26 17:07, Hayato Kuroda (Fujitsu) wrote:
>> Dear Fujii-san,
>>
>> Just in case - based on the agreement in [1], I updated patches to keep them
>> consistent. We can use same pictures for further discussions...
> 
> Thanks for updating the patches! I pushed them.

The buildfarm member "hake" reported a failure in the postgres_fdw regression test.

diff -U3 /export/home/elmer/c15x/buildroot/HEAD/pgsql.build/contrib/postgres_fdw/expected/postgres_fdw.out
/export/home/elmer/c15x/buildroot/HEAD/pgsql.build/contrib/postgres_fdw/results/postgres_fdw.out
--- /export/home/elmer/c15x/buildroot/HEAD/pgsql.build/contrib/postgres_fdw/expected/postgres_fdw.out    Fri Jul 26
19:16:292024
 
+++ /export/home/elmer/c15x/buildroot/HEAD/pgsql.build/contrib/postgres_fdw/results/postgres_fdw.out    Fri Jul 26
19:31:122024
 
@@ -12326,7 +12326,7 @@
    FROM postgres_fdw_get_connections(true);
   case
  ------
-    1
+    0
  (1 row)
  
  -- Clean up


The regression.diffs shows that pgfdw_conn_check returned 0 even though pgfdw_conn_checkable()
returned true. This can happen if the "revents" from poll() indicates something other than
POLLRDHUP. I think that "revents" could indicate POLLHUP, POLLERR, or POLLNVAL. Therefore,
IMO pgfdw_conn_check() should be updated as follows. I will test this change.

-               return (input_fd.revents & POLLRDHUP) ? 1 : 0;
+               return (input_fd.revents &
+                               (POLLRDHUP | POLLHUP | POLLERR | POLLNVAL)) ? 1 : 0;

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION



pgsql-hackers by date:

Previous
From: Masahiko Sawada
Date:
Subject: Re: Parallel heap vacuum
Next
From: Robert Haas
Date:
Subject: Re: tls 1.3: sending multiple tickets