[improve][build] Upgrade commons-compress version from 1.27.0 to 1.27.1 #24270
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
liangyepianzhou
approved these changes
May 8, 2025
15 tasks
dao-jun
approved these changes
May 8, 2025
1357d8f to
a06f6d6
Compare
This was referenced May 8, 2025
lhotari
approved these changes
May 8, 2025
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #24270 +/- ##
============================================
+ Coverage 73.57% 74.23% +0.66%
+ Complexity 32624 32218 -406
============================================
Files 1877 1866 -11
Lines 139502 145048 +5546
Branches 15299 16579 +1280
============================================
+ Hits 102638 107682 +5044
+ Misses 28908 28841 -67
- Partials 7956 8525 +569
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
manas-ctds
pushed a commit
to datastax/pulsar
that referenced
this pull request
May 14, 2025
….1 (apache#24270) (cherry picked from commit 0b00525) (cherry picked from commit 2e2d32a)
manas-ctds
pushed a commit
to datastax/pulsar
that referenced
this pull request
May 14, 2025
….1 (apache#24270) (cherry picked from commit 0b00525) (cherry picked from commit 2e2d32a)
srinath-ctds
pushed a commit
to datastax/pulsar
that referenced
this pull request
May 18, 2025
….1 (apache#24270) (cherry picked from commit 0b00525) (cherry picked from commit 2e2d32a)
KannarFr
pushed a commit
to CleverCloud/pulsar
that referenced
this pull request
Sep 22, 2025
walkinggo
pushed a commit
to walkinggo/pulsar
that referenced
this pull request
Oct 8, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
There are some issues with the version upgrade of commons-compress in the Pulsar and BookKeeper projects:
Upgrade Apache Commons libraries to compatible versions bookkeeper#4582 there are some issues with the Changes description of your PR, upgrade commons-compress from 1.26.0 to 1.27.1, not 1.27.0, I think the Changes can be modified
[improve] Upgrade Apache Commons library versions to compatible versions #24205 The version number is wrong,I have proposed a new fix [improve][build] Upgrade commons-compress version from 1.27.0 to 1.27.1 #24270 Please help review it
Motivation
commons-compress would be useful to upgrade to latest version,the new version brings improvements:
See release notes for details:
Modifications
Upgrade commons-compress version from 1.27.0 to 1.27.1
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes
Documentation
Check the box below or label this PR directly.
Need to update docs?
docdoc-requireddoc-not-neededdoc-completeMatching PR in forked repository
PR in forked repository: StevenLuMT#13