RFR: 8361249: PlainHttpConnection connection logic can be simplified [v3]
Daniel Fuchs
dfuchs at openjdk.org
Fri Jul 4 08:34:52 UTC 2025
> The PlainHttpConnection::connectAsync method implements a retry logic that will call connect() again if connect() throws the first time. This will not work, as the channel is closed when connect() throws. That logic should be removed. Reconnection should only be attempted at a higher level (MultiExchange).
Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
Review feedback
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26087/files
- new: https://git.openjdk.org/jdk/pull/26087/files/a63206df..12ba2924
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26087&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26087&range=01-02
Stats: 8 lines in 1 file changed: 0 ins; 8 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/26087.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26087/head:pull/26087
PR: https://git.openjdk.org/jdk/pull/26087
More information about the net-dev
mailing list