Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[improve][broker] PIP-293: Delete config disableBrokerInterceptors #20995

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

Technoboy-
Copy link
Contributor

PIP: #293

Documentation

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

@Technoboy- Technoboy- self-assigned this Aug 15, 2023
@Technoboy- Technoboy- added this to the 3.2.0 milestone Aug 15, 2023
@Technoboy- Technoboy- marked this pull request as ready for review August 15, 2023 10:39
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Aug 15, 2023
@Technoboy- Technoboy- closed this Aug 15, 2023
@Technoboy- Technoboy- reopened this Aug 15, 2023
@Technoboy- Technoboy- closed this Aug 21, 2023
@Technoboy- Technoboy- reopened this Aug 21, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 21, 2023

Codecov Report

Merging #20995 (50711e3) into master (63d9eaf) will increase coverage by 35.93%.
Report is 16 commits behind head on master.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #20995       +/-   ##
=============================================
+ Coverage     36.92%   72.85%   +35.93%     
- Complexity    12187    32299    +20112     
=============================================
  Files          1698     1875      +177     
  Lines        129846   141655    +11809     
  Branches      14163    15973     +1810     
=============================================
+ Hits          47947   103205    +55258     
+ Misses        75570    30237    -45333     
- Partials       6329     8213     +1884     
Flag Coverage Δ
inttests 24.76% <18.75%> (+0.61%) ⬆️
systests 25.30% <18.75%> (+0.15%) ⬆️
unittests 72.09% <100.00%> (+40.01%) ⬆️

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

Files Changed Coverage Δ
...org/apache/pulsar/broker/ServiceConfiguration.java 99.38% <ø> (+1.32%) ⬆️
.../java/org/apache/pulsar/broker/web/WebService.java 94.14% <100.00%> (+18.48%) ⬆️
...g/apache/pulsar/client/impl/RawBatchConverter.java 93.50% <100.00%> (+19.82%) ⬆️
...rg/apache/pulsar/compaction/TwoPhaseCompactor.java 74.88% <100.00%> (+4.70%) ⬆️

... and 1433 files with indirect coverage changes

@Technoboy- Technoboy- merged commit 0d25e9a into apache:master Aug 21, 2023
104 of 115 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/broker 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.

5 participants