RFR: 8368528: HttpClient.Builder.connectTimeout should accept arbitrarily large values [v6]
Volkan Yazici
vyazici at openjdk.org
Tue Nov 4 18:26:37 UTC 2025
On Tue, 4 Nov 2025 15:28:02 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Use `<p id="overflow">` in Javadoc as suggested in reviews
>
> src/java.net.http/share/classes/jdk/internal/net/http/common/Deadline.java line 39:
>
>> 37: * This class is immutable and thread-safe.
>> 38: * <p id="overflow">
>> 39: * Operations that add durations to a {@code Deadline}, whether represented as a
>
> Suggestion:
>
> * Operations that add or subtract durations to a {@code Deadline}, whether represented as a
Applied in 167d3b973c2.
> src/java.net.http/share/classes/jdk/internal/net/http/common/Deadline.java line 43:
>
>> 41: * nanoseconds) do not throw on numeric overflow if the resulting
>> 42: * {@code Deadline} would exceed {@link #MAX} or be less than {@link #MIN}.
>> 43: * Instead, {@code MAX} or {@code MIN} is returned, respectively.
>
> Suggestion:
>
> * Instead, {@code MAX} or {@code MIN} is returned, respectively.
> * Similarly, methods that return a duration as a {@code long} will either return
> * {@link Long#MAX_VALUE} or {@link Long#MIN_VALUE} if the returned quantity would
> * exceed the capacity of a {@code long}.
Applied in 167d3b973c2.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27973#discussion_r2491642861
PR Review Comment: https://git.openjdk.org/jdk/pull/27973#discussion_r2491643264
More information about the net-dev
mailing list