RFR: 8370024: HttpClient: QUIC congestion controller doesn't implement pacing [v2]

[Daniel Jeliński]

On Thu, 6 Nov 2025 11:07:42 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove unnecessary object allocation
>
> src/java.net.http/share/classes/jdk/internal/net/http/quic/QuicPacer.java line 177:
> 
>> 175:         long rttMicros = rttEstimator.state().smoothedRttMicros();
>> 176:         long cwnd = congestionController.congestionWindow();
>> 177:         return lastUpdate.plus(Duration.ofNanos(rttMicros
> 
> you could consider using the two args version of `plus` that can take ChronoUnit.MICROS.
> 
>     public Deadline plus(long amountToAdd, TemporalUnit unit) { ...

Good catch! Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28156#discussion_r2498791230