Skip to content

[fix][client] Fix lookup request semaphore not release problem#25038

Merged
congbobo184 merged 2 commits into
apache:masterfrom
congbobo184:fix/client/lookup_request_semaphore_release
Dec 5, 2025
Merged

[fix][client] Fix lookup request semaphore not release problem#25038
congbobo184 merged 2 commits into
apache:masterfrom
congbobo184:fix/client/lookup_request_semaphore_release

Conversation

@congbobo184

Copy link
Copy Markdown
Contributor

Motivation

fix bugs

Bug

if lookup request timeout, pendingLookupRequestSemaphore will not be released, so fix it

Modifications

Fix the bug.

Documentation

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

Matching PR in forked repository

PR in forked repository: congbobo184#23

@congbobo184 congbobo184 changed the title Fix/client/lookup request semaphore release [fix][client] Fix lookup request semaphore not release problem Dec 4, 2025
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Dec 4, 2025
@congbobo184 congbobo184 self-assigned this Dec 4, 2025
@congbobo184 congbobo184 added doc-label-missing java Pull requests that update Java code ready-to-test type/bug The PR fixed a bug or issue reported a bug labels Dec 4, 2025
@congbobo184 congbobo184 added this to the 4.2.0 milestone Dec 4, 2025
@lhotari

lhotari commented Dec 4, 2025

Copy link
Copy Markdown
Member

/pulsarbot rerun-failure-checks

@congbobo184 congbobo184 closed this Dec 5, 2025
@congbobo184 congbobo184 reopened this Dec 5, 2025
@codecov-commenter

codecov-commenter commented Dec 5, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.33%. Comparing base (b71bea4) to head (f8fca27).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #25038       +/-   ##
=============================================
+ Coverage     38.52%   74.33%   +35.81%     
- Complexity    13254    33745    +20491     
=============================================
  Files          1864     1921       +57     
  Lines        146187   150855     +4668     
  Branches      16977    17606      +629     
=============================================
+ Hits          56318   112143    +55825     
+ Misses        82183    29759    -52424     
- Partials       7686     8953     +1267     
Flag Coverage Δ
inttests 26.52% <0.00%> (+0.23%) ⬆️
systests 23.03% <0.00%> (+0.20%) ⬆️
unittests 73.85% <100.00%> (+39.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
.../java/org/apache/pulsar/client/impl/ClientCnx.java 69.01% <100.00%> (+17.81%) ⬆️

... and 1427 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@congbobo184 congbobo184 merged commit 9d8bf60 into apache:master Dec 5, 2025
170 of 183 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs java Pull requests that update Java code ready-to-test type/bug The PR fixed a bug or issue reported a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants