Skip to content

Comments

修改mapx.MergeFunc中预测capacity的方式#282

Closed
dxyinme wants to merge 1 commit intoecodeclub:devfrom
dxyinme:dxyinme/fix-map-merge-capacity
Closed

修改mapx.MergeFunc中预测capacity的方式#282
dxyinme wants to merge 1 commit intoecodeclub:devfrom
dxyinme:dxyinme/fix-map-merge-capacity

Conversation

@dxyinme
Copy link
Contributor

@dxyinme dxyinme commented May 18, 2025

No description provided.

@codecov
Copy link

codecov bot commented May 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.65%. Comparing base (0a2320e) to head (e1e71a2).
Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #282      +/-   ##
==========================================
- Coverage   94.73%   94.65%   -0.08%     
==========================================
  Files          75       75              
  Lines        3798     3800       +2     
==========================================
- Hits         3598     3597       -1     
- Misses        161      163       +2     
- Partials       39       40       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@flycash
Copy link
Contributor

flycash commented May 19, 2025

最糟糕的情况下就是全部都是独立的 key,合并不会有效果。你的改法还是有可能触发扩容。

@flycash flycash closed this May 19, 2025
@dxyinme
Copy link
Contributor Author

dxyinme commented May 19, 2025

我个人感觉为了回避扩容而多占空间是一个没有什么必要的事情,或者如果这么做,需要在注释里面标出来告诉用的人

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.

2 participants