> 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/58c78e57..a63206df Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26087&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26087&range=00-01 Stats: 14 lines in 1 file changed: 2 ins; 6 del; 6 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
