Skip to content

Conversation

@PansonPanson
Copy link
Contributor

Avoiding multiple resizing by replacing iterative put with putAll .
image

@PansonPanson PansonPanson changed the title avoid multiple resizing [ISSUE #4685] Avoiding multiple resizing by replacing iterative put with putAll . Jul 24, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 25, 2022

Codecov Report

Merging #4684 (d852289) into develop (efbc4a1) will increase coverage by 0.40%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #4684      +/-   ##
=============================================
+ Coverage      44.32%   44.72%   +0.40%     
- Complexity      7282     7521     +239     
=============================================
  Files            968      975       +7     
  Lines          65899    67380    +1481     
  Branches        8635     8891     +256     
=============================================
+ Hits           29208    30139     +931     
- Misses         33085    33524     +439     
- Partials        3606     3717     +111     
Impacted Files Coverage Δ
...che/rocketmq/common/protocol/route/BrokerData.java 33.33% <0.00%> (+1.07%) ⬆️
...va/org/apache/rocketmq/store/FlushDiskWatcher.java 81.25% <0.00%> (-9.38%) ⬇️
...ava/org/apache/rocketmq/broker/util/HookUtils.java 12.28% <0.00%> (-4.87%) ⬇️
...va/org/apache/rocketmq/common/message/Message.java 52.94% <0.00%> (-4.03%) ⬇️
...a/org/apache/rocketmq/filter/util/BloomFilter.java 60.43% <0.00%> (-2.20%) ⬇️
...he/rocketmq/broker/processor/PopReviveService.java 24.90% <0.00%> (-1.81%) ⬇️
.../rocketmq/broker/filter/ConsumerFilterManager.java 72.19% <0.00%> (-1.80%) ⬇️
...mq/store/ha/autoswitch/AutoSwitchHAConnection.java 73.56% <0.00%> (-1.37%) ⬇️
...a/org/apache/rocketmq/broker/BrokerController.java 46.10% <0.00%> (-1.16%) ⬇️
...a/org/apache/rocketmq/store/StoreStatsService.java 49.72% <0.00%> (-0.56%) ⬇️
... and 38 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 48.96% when pulling d852289 on PansonPanson:develop_20220720 into a6ca01a on apache:develop.

Copy link
Member

@Oliverwqcwrw Oliverwqcwrw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @PansonPanson ,
thanks for your contribution, Please resolve conflicts

@PansonPanson
Copy link
Contributor Author

Hi,@Oliverwqcwrw @tsunghanjacktsai
This code snippet has been carefully profiled and optimized. I'll close this issue.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants