On Wed, 15 Jul 2026 11:44:18 GMT, Daniel Jeliński <[email protected]> wrote:

> Change the tests to use blocking sockets instead.
> 
> This change removes the busy-looping on finishConnect, read, and write. The 
> change is safe because:
> - finishConnect is no longer necessary,
> - all uses of write() loop until wither all data is written or an error occurs
> - read() is only used when additional data is expected.
> 
> Tier2 still green.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Looks good to me, but do you think there will be another commit before merge? 
If so, could you please replace all wildcard imports. 

IMO not worth another commit if there is no other changes

-------------

Marked as reviewed by myankelevich (Committer).

PR Review: https://git.openjdk.org/jdk/pull/31912#pullrequestreview-4779122586

Reply via email to