[jdk21u-dev] RFR: 8342075: HttpClient: improve HTTP/2 flow control checks [v3]

Alexey Bakhtin abakhtin at openjdk.org
Thu Mar 20 15:45:13 UTC 2025


On Tue, 18 Mar 2025 22:19:10 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:

>> Alexey Bakhtin 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 10 additional commits since the last revision:
>> 
>>  - Merge master
>>  - Add Http2TestExchange::getConnectionKey() from JDK-8335181
>>  - Backport b0ac633b2d0076d64b463b2a6ce19abf6b12c50f
>>  - Revert "Backport fc98998627443d6e73ac70661f47f48b30525712"
>>    
>>    This reverts commit 899abf76e7a381abdfbc30f08a6a5678c0a8065d.
>>  - Revert "Backport 720b44648bcff997278af92746f942b2425298a5"
>>    
>>    This reverts commit a5b7bb4d432c868d598c35f54306e40747071ad7.
>>  - Revert "Remove extra import"
>>    
>>    This reverts commit 480336754274ea62ab2e9b076dcec91f7d483c25.
>>  - Backport 6273ab97dc1a0d3c1f51ba94694d9594dd7593d4
>>  - Remove extra import
>>  - Backport 720b44648bcff997278af92746f942b2425298a5
>>  - Backport fc98998627443d6e73ac70661f47f48b30525712
>
>> At the time of backport creation, JDK-8315181 was not integrated into Oracle 21.0.8 yet. This is why I decided to make a patch without it, especially because JDK-8315181 is required for tests only. Now JDK-8315181 is integrated into Oracle and I reopened #1416 & #1417 backports. I will rebase this patch as soon as #1417 is accepted
> 
> Thanks. Sorry for messing you about, but if we're going to have those two in 21u, I think it makes sense to do them in order.

@gnu-andrew 
I have rebased the PR to the latest master. 
The backport is almost clean now, except for the import section in Http2Connection.java. This section was added by JDK-8310330 in JDK22 but not backported.
I have removed test changes from the initial description as soon as they are in the master already.
All related jtreg tests are passed

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/1428#issuecomment-2740897904


More information about the jdk-updates-dev mailing list