You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "codecov-commenter (via GitHub)" <gi...@apache.org> on 2023/06/29 02:37:38 UTC

[GitHub] [pulsar] codecov-commenter commented on pull request #20647: [fix][broker] Fix the publish latency spike from the contention of MessageDeduplication

codecov-commenter commented on PR #20647:
URL: https://github.com/apache/pulsar/pull/20647#issuecomment-1612354025

   ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/20647?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   > Merging [#20647](https://app.codecov.io/gh/apache/pulsar/pull/20647?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (b0cebfd) into [master](https://app.codecov.io/gh/apache/pulsar/commit/2b01f83ea4f3ab7b22f4ea2b0290cc183e79bbb8?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (2b01f83) will **increase** coverage by `39.54%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree graph](https://app.codecov.io/gh/apache/pulsar/pull/20647/graphs/tree.svg?width=650&height=150&src=pr&token=acYqCpsK9J&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)](https://app.codecov.io/gh/apache/pulsar/pull/20647?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   
   ```diff
   @@              Coverage Diff              @@
   ##             master   #20647       +/-   ##
   =============================================
   + Coverage     33.58%   73.12%   +39.54%     
   - Complexity    12127    32016    +19889     
   =============================================
     Files          1613     1867      +254     
     Lines        126241   138683    +12442     
     Branches      13770    15240     +1470     
   =============================================
   + Hits          42396   101410    +59014     
   + Misses        78331    29249    -49082     
   - Partials       5514     8024     +2510     
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | inttests | `24.18% <50.00%> (-0.01%)` | :arrow_down: |
   | systests | `25.02% <50.00%> (?)` | |
   | unittests | `72.36% <100.00%> (+40.32%)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   | [Impacted Files](https://app.codecov.io/gh/apache/pulsar/pull/20647?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [...roker/service/persistent/MessageDeduplication.java](https://app.codecov.io/gh/apache/pulsar/pull/20647?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Jyb2tlci9zZXJ2aWNlL3BlcnNpc3RlbnQvTWVzc2FnZURlZHVwbGljYXRpb24uamF2YQ==) | `82.53% <100.00%> (+42.35%)` | :arrow_up: |
   
   ... and [1520 files with indirect coverage changes](https://app.codecov.io/gh/apache/pulsar/pull/20647/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org