Skip to content

Clean up store stats#7687

Merged
HaoranYi merged 1 commit intoanza-xyz:masterfrom
HaoranYi:clean_store_stat
Aug 25, 2025
Merged

Clean up store stats#7687
HaoranYi merged 1 commit intoanza-xyz:masterfrom
HaoranYi:clean_store_stat

Conversation

@HaoranYi
Copy link
Copy Markdown

Problem

"get_slot_store" and "find_existing_store" stats are osolete since we no longer do "Append" for ancient appendvec.

Summary of Changes

Remove them.

Fixes #

@mergify
Copy link
Copy Markdown

mergify Bot commented Aug 22, 2025

⚠️ The sha of the head commit of this PR conflicts with #7624. Mergify cannot evaluate rules on this PR. ⚠️

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.4%. Comparing base (94eb488) to head (ddfaddc).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #7687     +/-   ##
=========================================
- Coverage    83.4%    83.4%   -0.1%     
=========================================
  Files         813      814      +1     
  Lines      366220   366392    +172     
=========================================
+ Hits       305691   305739     +48     
- Misses      60529    60653    +124     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…longer do "Append" for ancient appendvec.

so remove them.

-    pub store_get_slot_store: AtomicU64,
-    pub store_find_existing: AtomicU64,
@HaoranYi HaoranYi changed the title clean store stat Clean up store stats Aug 25, 2025
@HaoranYi HaoranYi merged commit 8b10f19 into anza-xyz:master Aug 25, 2025
43 checks passed
@HaoranYi HaoranYi deleted the clean_store_stat branch August 25, 2025 17:50
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.

3 participants