Skip to content

Conversation

@StevenLuMT
Copy link
Owner

@StevenLuMT StevenLuMT commented May 8, 2025

Background

There are some issues with the version upgrade of commons-compress in the Pulsar and BookKeeper projects:

  1. Upgrade Apache Commons libraries to compatible versions apache/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

  2. [improve] Upgrade Apache Commons library versions to compatible versions apache/pulsar#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 apache/pulsar#24270 Please help review it

Motivation

commons-compress would be useful to upgrade to latest version,the new version brings improvements:

  1. Compression into BZip2 format has unexpected end of file when using a BufferedOutputStream. Fixes COMPRESS-686.

See release notes for details:

image

Modifications

Upgrade commons-compress version from 1.27.0 to 1.27.1

Verifying this change

  • Make sure that the change passes the CI checks.
    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

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

Check the box below or label this PR directly.

Need to update docs?

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: #13

@StevenLuMT StevenLuMT changed the title commons-compress version from 1.27.0 to 1.27.1 [improve][build] Upgrade commons-compress version from 1.27.0 to 1.27.1 May 8, 2025
@StevenLuMT StevenLuMT force-pushed the master-upgradeCommonsCompress branch from 1357d8f to a06f6d6 Compare May 8, 2025 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants