On Fri, 24 Oct 2025 06:06:44 GMT, Josiah Noel <[email protected]> wrote:

>> Now ExchangeImpl will default to having a separate attribute map for the 
>> request duration.
>
> Josiah Noel has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update ExchangeAttributeTest.java

Thank you for the update, this looks good to me. I'll let Michael (and Daniel) 
have a chance to review the updated version.

On a general note, since you are contributing to the `HttpServer` area, I would 
also recommend that you run `tier2` tests locally. The GitHub actions job for 
the JDK only runs tier1 and that tier doesn't cover the networking area.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27652#pullrequestreview-3374705582

Reply via email to