On Mon, 9 Feb 2026 21:59:59 GMT, Daniel Fuchs <[email protected]> wrote:
>> This change adds a sentence to the the description of the three keepalive >> properties to describe the effect of setting a value <= 0: >> >>> If the provided value is negative, the default value is used. A value of 0 >>> causes the timeout to fire immediately. >> >> I also took this opportunity to add a link from `module-info.java` to the >> `java.net.http` package description, and an `@see` link from the >> `java.net.http` package description to the `module-info`. >> >> This is a doc change only - there is no code or behaviour change. Since it >> only affects jdk specific properties and is just a clarification of >> pre-existing behavior I believe no CSR is required. Please comment if you >> believe writing a CSR would be better. > > Daniel Fuchs has updated the pull request incrementally with one additional > commit since the last revision: > > Improve the wording Hello Daniel, the changes look good to me. > Please comment if you believe writing a CSR would be better. In the past any assertable behavioural specification has been backed by a CSR, so I think it would be good to create one for this change. ------------- Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/29587#pullrequestreview-3778207930
