RFR: 8296804: Document HttpClient configuration properties in java.net.http module-info [v3]

Michael McMahon michaelm at openjdk.org
Mon Nov 21 16:14:23 UTC 2022


On Mon, 21 Nov 2022 13:36:29 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Michael McMahon has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - Merge branch 'master' into http.properties
>>  - update after Daniel's review
>>  - first draft
>
> src/java.net.http/share/classes/module-info.java line 113:
> 
>> 111:  * <li><p><b>{@systemProperty jdk.httpclient.maxstreams}</b> (default: 100)<br>
>> 112:  * The maximum number of HTTP/2 streams per connection.
>> 113:  * </li>
> 
> It might be useful to be a bit more precise here: this is the maximum number of concurrent push streams that a server is allowed to initiate on an HTTP/2 connection

Right, good point. It's the client telling the server how many streams it is allowed to open. So, that implies push streams only

> src/java.net.http/share/classes/module-info.java line 133:
> 
>> 131:  * This allows the thread to terminate when no longer needed. The value provided is limited
>> 132:  * to the range from 1 to 1200(???) seconds.
>> 133:  * </li>
> 
> Do we need to document this property? If yes then what is the meaning of the `(???)` question marks?

The question mark was a place holder to query the value range. 1200 seemed extremely high. But, I think we don't need to document it at all.

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

PR: https://git.openjdk.org/jdk/pull/11241


More information about the net-dev mailing list