Re: Timeout parameters - Mailing list pgsql-hackers

From Kyotaro HORIGUCHI
Subject Re: Timeout parameters
Date
Msg-id 20190329.164421.193524577.horiguchi.kyotaro@lab.ntt.co.jp
Whole thread Raw
In response to RE: Timeout parameters  ("Nagaura, Ryohei" <nagaura.ryohei@jp.fujitsu.com>)
Responses RE: Timeout parameters
List pgsql-hackers
Hi, thank you for the new version.

This compiles on Windows. (I didn't comfirmed that it works
correctly..)

# ȁ  (I inserted this garbage to force my mailer to send in utf-8).

At Fri, 29 Mar 2019 06:52:41 +0000, "Nagaura, Ryohei" <nagaura.ryohei@jp.fujitsu.com> wrote in
<EDA4195584F5064680D8130B1CA91C4540F81D@G01JPEXMBYT04>
> Hi all.
> 
> I found my mistake in backend patch.
> 
> I modified from
> +    port->keepalives_count = count;
> to
> +    port->tcp_user_timeout = timeout;
> in line 113.

I hadn't noticed^^;

It look good, but still I have some comments.

+        Specifies the number of milliseconds that transmitted data may
+        remain unacknowledged before a connection is forcefully closed.

Hmm. "forcefully" means powerful or assertive, in Japanese "力強く
" or "強硬に".  "forcibly" means acoomplished through force, in
Japanesee "無理やり" or "強引に". Actually the latter is used in
man 7 tcp.

http://man7.org/linux/man-pages/man7/tcp.7.html
>  time in milliseconds that transmitted data may remain
>  unacknowledged before TCP will forcibly close the
>  corresponding connection and return ETIMEDOUT to the

+#tcp_user_timeout = 0        # TCP_USER_TIMEOUT, in milliseconds;

The comment protrudes left.

+# - TCP USER TIMEOUT -

Just above in the sample file, there are lines like this:

> # - TCP Keepalives -
> # see "man 7 tcp" for details
> 
> #tcp_keepalives_idle = 0        # TCP_KEEPIDLE, in seconds;

Couldn't we have the new variable in the section, like this?

-# - TCP Keepalives -
+# - TCP timeout settings -
 # see "man 7 tcp" for details
..
 #tcp_keepalives_count = 0        # TCP_KEEPCNT;
                     # 0 selects the system default
+#tcp_user_timeout = 0            # TCP_USER_TIMEOUT, in milliseconds;
                    # 0 selects the system default


regars.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center

pgsql-hackers by date:

Previous
From: Sergei Kornilov
Date:
Subject: Re: REINDEX CONCURRENTLY 2.0
Next
From: Michael Paquier
Date:
Subject: Re: REINDEX CONCURRENTLY 2.0