8381670: Revert the changes to GZIPInputStream related to InputStream.available() usage#2853
8381670: Revert the changes to GZIPInputStream related to InputStream.available() usage#2853jyma22 wants to merge 1 commit into
Conversation
….available() usage
|
👋 Welcome back jyma22! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
|
/approval request for backport of JDK-8381670: Revert the changes to GZIPInputStream related to InputStream.available() usage. This fixes a performance regression in GZIPInputStream and the CSR was approved. This pull request is not clean because the tip reverted 2 commits and only one needed to be reverted in 21 |
|
This backport pull request has now been updated with issue from the original commit. |
|
@jyma22 This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a |
|
/keepalive |
|
@jyma22 The pull request is being re-evaluated and the inactivity timeout has been reset. |
|
@jyma22 This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a |
Backporting JDK-8381670: Revert the changes to GZIPInputStream related to InputStream.available() usage.
This reverts commit fcbf863f4b7 due to performance regression in GZIPInputStream.
Tested on x64 with
make test TEST=test/jdk/java/util/zip/GZIPProgress
Issues
Reviewers
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk21u-dev.git pull/2853/head:pull/2853$ git checkout pull/2853Update a local copy of the PR:
$ git checkout pull/2853$ git pull https://git.openjdk.org/jdk21u-dev.git pull/2853/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 2853View PR using the GUI difftool:
$ git pr show -t 2853Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk21u-dev/pull/2853.diff
Using Webrev
Link to Webrev Comment