On Fri, 21 Feb 2025 13:02:25 GMT, Daniel Fuchs <[email protected]> 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