[improve][broker] Upgrade bookkeeper to 4.17.4 - #26211
Closed
StevenLuMT wants to merge 201 commits into
Closed
Conversation
(cherry picked from commit 260c386)
…gement (apache#25400) (cherry picked from commit f70745c)
…api (apache#25478) (cherry picked from commit f72a5a7)
…evel (apache#25483) (cherry picked from commit 0b73b3a)
(cherry picked from commit 5b2778e)
…pache#25520) (cherry picked from commit f688ff7)
…solution with fallback to localhost (apache#25238)" (apache#25523) (cherry picked from commit 352147b)
(cherry picked from commit 4c96c73)
Signed-off-by: Lari Hotari <lhotari@apache.org>
…k devices (apache#25510) (cherry picked from commit 089f87f)
…6-34478, CVE-2026-34480, CVE-2026-34481 Upgrades org.apache.logging.log4j:* from 2.25.3 to 2.25.4 to fix: - CVE-2026-34477: verifyHostName attribute silently ignored in TLS configuration - CVE-2026-34478: log injection in Rfc5424Layout due to silent configuration incompatibility - CVE-2026-34480: XmlLayout fails to sanitize characters forbidden by XML 1.0 - CVE-2026-34481: JsonTemplateLayout produces invalid JSON for non-finite floats (cherry picked from commit dfb06f1)
(cherry picked from commit e05c212)
(cherry picked from commit 865e276)
Co-authored-by: Lari Hotari <lhotari@users.noreply.github.com> (cherry picked from commit 20a380a)
(apache#25546) (cherry picked from commit a1613bc)
…on cache lookup (apache#25367) ### Motivation `TopicName.get()` previously used `ConcurrentHashMap.computeIfAbsent()` to populate the topic-name cache. Although `computeIfAbsent` is atomic, it holds the internal bin-lock for the entire duration of the mapping function, which includes the non-trivial `TopicName` construction (string splitting, validation, etc.). Under high-concurrency workloads where many threads simultaneously encounter the same uncached topic name, this causes unnecessary lock contention and can degrade throughput. ### Modifications Replace `computeIfAbsent` with an explicit two-step pattern: 1. **Fast path**: call `cache.get(topic)` first — a single volatile read with no locking — and return immediately on a cache hit (steady-state case). 2. **Slow path** (cache miss): construct `TopicName` *outside* the lock, then use `cache.put()` to insert. (cherry picked from commit 8c4e83d)
…pache#24463) (cherry picked from commit 3130a93)
…ache#25272) Co-authored-by: 张浩 <zhanghao60@100.me> (cherry picked from commit 8e37e3e)
…sters policy (apache#25551) (cherry picked from commit 9033223)
…e#25198 exclusion - lz4 compression with Kafka would break without this change
…try (apache#25460) (cherry picked from commit 56442c2)
…teSameNamedProducerShouldFail (apache#25497) (cherry picked from commit 7ce8a8c)
…che#25427) (cherry picked from commit 0c285f4)
…ests: PulsarClientException$AlreadyClosedException: Client already closed (apache#25509) (cherry picked from commit bf45974)
…pache#25500) (cherry picked from commit 12bcd6b)
…y to DEBUG (apache#25558) (cherry picked from commit 423a746)
….18.8 InetSocketAddress deserialization
…ache#25645) (cherry picked from commit c3fde12)
…flight read (apache#26106) (cherry picked from commit baf2211)
…rrent cache-init reload (apache#26132) (cherry picked from commit decc80f)
…che#26122) (cherry picked from commit 116212f)
… Mockito stubbing (apache#26083) (cherry picked from commit aa2d1bc)
…ed executor (apache#26123) (cherry picked from commit 12d8aa9)
…d gate the policy replay (apache#26134) (cherry picked from commit 0629dc5)
(cherry picked from commit 1b3e9ec)
… also address CVEs (apache#26140) (cherry picked from commit 82a5cdf)
…untime manager (apache#26136) Signed-off-by: Dream95 <zhou_8621@163.com> (cherry picked from commit 7a40ada)
…/pulsar-function-go (apache#26142) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Lari Hotari <lhotari@apache.org> (cherry picked from commit 055882b)
…ata-store URIs (apache#26150) Co-authored-by: void-ptr974 <void-ptr974@protonmail.com>
… metadata backend (apache#26158) Co-authored-by: ievgenpolozhenkov <ievgenpolozhenkov@geotab.com> (cherry picked from commit 8d5cac6) Assisted-by: Claude Code (Opus 4.8)
…for Etcd - apache#26158 hasn't been implemented for Etcd
…ache#26188) (cherry picked from commit 16476b5)
Contributor
|
@StevenLuMT BK is already at 4.18 in master branch |
Member
|
@StevenLuMT please create a PR for branch-4.2 |
Member
Author
Got it |
StevenLuMT
force-pushed
the
upgrade-bookkeeper-4.17.4
branch
from
July 21, 2026 05:16
031f580 to
64fc12b
Compare
Member
Author
|
new created pr: #26219 |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 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
Upgrade bookkeeper to 4.17.4
Modifications
Upgrade bookkeeper to 4.17.4
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#17