Hi Nagaura-san, Your socket_timeout patch still does not apply either with git or patch command. It says it's still corrupted. I'm not sure about the workaround, because the --ignore-space-change and --ignore-whitespace did not work for me. Maybe it might have something to do with your editor when creating the patch. Could you confirm? The CFbot is also another way to check if your latest patches work. http://commitfest.cputube.org/
On Tuesday, February 26, 2019 5:16PM (GMT+9), Nagaura, Ryohei wrote: > "communication inactivity" seems to be a little extreme. > If the communication layer is truly dead you will use keepalive. > This use case is when socket option is not available for some reason. > So it would be better "terminating the connection" in my thought. About the doc. Alright. - There's a typo in the doc: connectoin --> connection - In the code, there's a part where between 0-2 seconds is interpreted as 2 because it's the minimum time. I think the comment should be improved, and use insist (force) instead of "need". This should also be added in the documentation. From: /* * Rounding could cause communication to fail; need at least 2 secs */ To: /* * Rounding could cause communication to fail; * insist on at least two seconds. */ Docs - added how it should be written as decimal integer, similar to connect_timeout - used closing instead of terminating, to be consistent with other params - added the minimum allowed timeout socket_timeout Controls the number of second (write as a decimal integer, e.g. 10) of client's waiting time for individual socket read/write operations before closing the connection. This can be used both as a force global query timeout and network problems detector. A value of zero (the default) turns this off, which means wait indefinitely. The minimum allowed timeout is 2 seconds, so a value of 1 is interpreted as 2. > About TCP_USER_TIMEOUT patches, there are only miscellaneous changes: > removing trailing spaces and making comments of parameters lower case > as you pointed out. Thanks for the update. Confirmed the change. Regards, Kirk Jamison
