Skip to content

Conversation

@StevenLuMT
Copy link
Owner

@StevenLuMT StevenLuMT commented Jun 26, 2025

Background

Upgrade bookkeeper to 4.17.2

Modifications

  1. [improve][broker] Upgrade bookkeeper to 4.17.2

  2. [tmp-prepare-release] add the respository source for bk-staging(this commit will be deleted later)

  3. [improve][build] Migrate deprecated commons-configuration 1.x to commons-configuration 2.x, base on Migrate deprecated commons-configuration 1.x to commons-configuration 2.x apache/bookkeeper#4604
    3.1. import change: org.apache.commons.lang to org.apache.commons.lang3
    3.2. import change: org.apache.commons.configuration to org.apache.commons.configuration2
    3.3. impl change: setDelimiterParsingDisabled(true) to setListDelimiterHandler(new DisabledListDelimiterHandler())

  4. Enable ZooKeeper client to establish connection in read-only mode, base on Enable ZooKeeper client to establish connection in read-only mode apache/bookkeeper#4244

  5. Upgrade gRPC to 1.72.0 : grpc version match to bookeeper 4.17.2, base on Upgrade to grpc 1.72.0 apache/bookkeeper#4591

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: #15

@StevenLuMT StevenLuMT force-pushed the master-upgradeBookie4.17.2 branch from dee7dee to 5902d5f Compare July 3, 2025 01:34
@StevenLuMT StevenLuMT force-pushed the master-upgradeBookie4.17.2 branch from 5902d5f to 2d025b0 Compare July 4, 2025 06:23
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