On Thu, 27 Aug 2026 12:28:05 GMT, Lee Jiwon <[email protected]> wrote:

>> Replaces the fixed delay in `IsAvailable.java` with reads from the socket 
>> used by `HttpClient::available`.
>> The tests wait for EOF or data on that socket before invoking the existing 
>> `HttpClient::available` assertion.
>> 
>> No production code is changed.
>> 
>> Testing:
>> - `make test TEST='test/jdk/sun/net/www/http/HttpClient/IsAvailable.java' 
>> JTREG='REPEAT_COUNT=200;VERBOSE=summary;RETAIN=fail'` (200/200 passed)
>> - `make test TEST='test/jdk/sun/net/www/http/HttpClient/IsAvailable.java' 
>> JTREG='VERBOSE=summary;RETAIN=fail;VM_OPTIONS=-Dsun.net.client.defaultReadTimeout=200'`
>>  (passed)
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Lee Jiwon has refreshed the contents of this pull request, and previous 
> commits have been removed. The incremental views will show differences 
> compared to the previous content of the PR. The pull request contains one new 
> commit since the last revision:
> 
>   8390064: Stabilize IsAvailable test

The approach taken here seems reasonable to me.

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

PR Comment: https://git.openjdk.org/jdk/pull/32539#issuecomment-5450825040

Reply via email to