Skip to content

Conversation

@poorbarcode
Copy link
Contributor

Motivation

branch-4.0 has such an issue, which occurs as follows. I have pushed a PR to fix it #24680

  • Create a partitioned topic with 2 partitions
  • All partitions are deleted because the feature brokerDeleteInactiveTopicsEnabled is true
  • Try to get the topic-level policies; you will get a Topic not found error

The master branch does not have the issue because #24225 have fixed it here

This PR is in order to add the same test as #24680

Modifications

Add a new test PartitionCreationTest.testGetPoliciesIfPartitionsNotCreated

Documentation

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

Matching PR in forked repository

PR in forked repository: x

@poorbarcode poorbarcode added this to the 4.1.0 milestone Aug 28, 2025
@poorbarcode poorbarcode self-assigned this Aug 28, 2025
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Aug 28, 2025
@poorbarcode
Copy link
Contributor Author

/pulsarbot rerun-failure-checks

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.20%. Comparing base (490ba0c) to head (4e8d439).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #24681       +/-   ##
=============================================
+ Coverage     38.72%   74.20%   +35.47%     
- Complexity    13076    33383    +20307     
=============================================
  Files          1836     1893       +57     
  Lines        143697   147772     +4075     
  Branches      16616    17089      +473     
=============================================
+ Hits          55650   109655    +54005     
+ Misses        80600    29360    -51240     
- Partials       7447     8757     +1310     
Flag Coverage Δ
inttests 26.56% <ø> (-0.08%) ⬇️
systests 22.73% <ø> (+0.01%) ⬆️
unittests 73.69% <ø> (+39.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1391 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.

@Technoboy- Technoboy- merged commit 03e0787 into apache:master Aug 29, 2025
50 checks passed
KannarFr pushed a commit to CleverCloud/pulsar that referenced this pull request Sep 22, 2025
walkinggo pushed a commit to walkinggo/pulsar that referenced this pull request Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test 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