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/08 08:07:22 UTC

[GitHub] [pulsar] codecov-commenter commented on pull request #20516: [improve][admin] Pretty print bookies racks-placement command output

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

   ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   > Merging [#20516](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (cd1b036) into [master](https://app.codecov.io/gh/apache/pulsar/commit/3b862ae614fae795e3312c0b298cc3c2b33a698f?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (3b862ae) will **increase** coverage by `36.11%`.
   > The diff coverage is `61.84%`.
   
   [![Impacted file tree graph](https://app.codecov.io/gh/apache/pulsar/pull/20516/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/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   
   ```diff
   @@              Coverage Diff              @@
   ##             master   #20516       +/-   ##
   =============================================
   + Coverage     36.78%   72.90%   +36.11%     
   - Complexity    12059    31917    +19858     
   =============================================
     Files          1690     1867      +177     
     Lines        129001   138565     +9564     
     Branches      14041    15219     +1178     
   =============================================
   + Hits          47453   101017    +53564     
   + Misses        75294    29525    -45769     
   - Partials       6254     8023     +1769     
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | inttests | `24.16% <14.47%> (+0.02%)` | :arrow_up: |
   | systests | `24.90% <0.00%> (-0.06%)` | :arrow_down: |
   | unittests | `72.20% <61.84%> (+40.25%)` | :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/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [.../apache/pulsar/broker/admin/impl/PackagesBase.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Jyb2tlci9hZG1pbi9pbXBsL1BhY2thZ2VzQmFzZS5qYXZh) | `76.19% <ø> (+23.89%)` | :arrow_up: |
   | [...rg/apache/pulsar/broker/service/BrokerService.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Jyb2tlci9zZXJ2aWNlL0Jyb2tlclNlcnZpY2UuamF2YQ==) | `80.59% <ø> (+24.26%)` | :arrow_up: |
   | [.../apache/pulsar/functions/instance/ContextImpl.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWZ1bmN0aW9ucy9pbnN0YW5jZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Z1bmN0aW9ucy9pbnN0YW5jZS9Db250ZXh0SW1wbC5qYXZh) | `61.00% <0.00%> (+18.41%)` | :arrow_up: |
   | [...s/runtime/kubernetes/KubernetesRuntimeFactory.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWZ1bmN0aW9ucy9ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9wdWxzYXIvZnVuY3Rpb25zL3J1bnRpbWUva3ViZXJuZXRlcy9LdWJlcm5ldGVzUnVudGltZUZhY3RvcnkuamF2YQ==) | `80.21% <50.00%> (+80.21%)` | :arrow_up: |
   | [...pache/pulsar/broker/admin/impl/NamespacesBase.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Jyb2tlci9hZG1pbi9pbXBsL05hbWVzcGFjZXNCYXNlLmphdmE=) | `72.05% <56.86%> (+41.36%)` | :arrow_up: |
   | [...n/java/org/apache/pulsar/admin/cli/CliCommand.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWNsaWVudC10b29scy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2FkbWluL2NsaS9DbGlDb21tYW5kLmphdmE=) | `59.55% <66.66%> (+59.55%)` | :arrow_up: |
   | [...ker/authorization/PulsarAuthorizationProvider.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3B1bHNhci9icm9rZXIvYXV0aG9yaXphdGlvbi9QdWxzYXJBdXRob3JpemF0aW9uUHJvdmlkZXIuamF2YQ==) | `71.42% <100.00%> (+39.74%)` | :arrow_up: |
   | [.../org/apache/pulsar/broker/admin/AdminResource.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Jyb2tlci9hZG1pbi9BZG1pblJlc291cmNlLmphdmE=) | `78.85% <100.00%> (+35.05%)` | :arrow_up: |
   | [...pulsar/broker/admin/impl/PersistentTopicsBase.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWJyb2tlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2Jyb2tlci9hZG1pbi9pbXBsL1BlcnNpc3RlbnRUb3BpY3NCYXNlLmphdmE=) | `63.91% <100.00%> (+53.73%)` | :arrow_up: |
   | [...n/java/org/apache/pulsar/admin/cli/CmdBookies.java](https://app.codecov.io/gh/apache/pulsar/pull/20516?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cHVsc2FyLWNsaWVudC10b29scy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcHVsc2FyL2FkbWluL2NsaS9DbWRCb29raWVzLmphdmE=) | `87.50% <100.00%> (+87.50%)` | :arrow_up: |
   
   ... and [1426 files with indirect coverage changes](https://app.codecov.io/gh/apache/pulsar/pull/20516/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