Skip to content

[fix][build] Fix compilation issues for PR #25219 changes after merging conflicting PR #25278 - #25300

Merged
poorbarcode merged 1 commit into
apache:masterfrom
lhotari:lh-fix-broken-build-2026-03-09
Mar 9, 2026
Merged

[fix][build] Fix compilation issues for PR #25219 changes after merging conflicting PR #25278#25300
poorbarcode merged 1 commit into
apache:masterfrom
lhotari:lh-fix-broken-build-2026-03-09

Conversation

@lhotari

@lhotari lhotari commented Mar 9, 2026

Copy link
Copy Markdown
Member

Motivation & Modifications

Fix compilation issues for PR #25219 changes after merging conflicting PR #25278

Documentation

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

@lhotari
lhotari requested a review from Technoboy- March 9, 2026 12:53
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Mar 9, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.57%. Comparing base (e902c68) to head (d89f369).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
...apache/pulsar/metadata/impl/DualMetadataStore.java 80.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #25300       +/-   ##
=============================================
+ Coverage     37.15%   72.57%   +35.41%     
- Complexity    13284    34571    +21287     
=============================================
  Files          1911     1967       +56     
  Lines        151956   156121     +4165     
  Branches      17297    17784      +487     
=============================================
+ Hits          56460   113299    +56839     
+ Misses        87798    33764    -54034     
- Partials       7698     9058     +1360     
Flag Coverage Δ
inttests 25.60% <0.00%> (+0.06%) ⬆️
systests 22.34% <0.00%> (+0.05%) ⬆️
unittests 73.53% <90.00%> (+39.53%) ⬆️

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

Files with missing lines Coverage Δ
...tadata/coordination/impl/MigrationCoordinator.java 74.75% <100.00%> (+74.75%) ⬆️
...apache/pulsar/metadata/impl/DualMetadataStore.java 80.00% <80.00%> (+38.82%) ⬆️

... and 1426 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.

@poorbarcode
poorbarcode merged commit c70f263 into apache:master Mar 9, 2026
56 of 57 checks passed
@lhotari lhotari added this to the 4.2.0 milestone Mar 24, 2026
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 ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants