On Fri, 21 Feb 2025 13:02:25 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> src/java.net.http/share/classes/jdk/internal/net/http/Stream.java line 1124:
>> 
>>> 1122:             if (debug.on())
>>> 1123:                 debug.log("RequestSubscriber: onSubscribe, request 
>>> 1");
>>> 1124:             
>>> exchange.executor().safeDelegate().execute(this::tryRequestMore);
>> 
>> I'd appreciate it if we can document the need to execute this asynchronously 
>> using this particular executor – while it is not ultimate measure, 
>> nevertheless, it is not apparent to me. 🙈
>
> PostFromGet will/might get wedged if we execute inline at this point. See my 
> previous explanations.

OK - I have added some comments.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23716#discussion_r1971451241

Reply via email to