Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

cleanup accounts shrink debug message#13089

Merged
sakridge merged 1 commit intosolana-labs:masterfrom
sakridge:shrink-debug
Oct 23, 2020
Merged

cleanup accounts shrink debug message#13089
sakridge merged 1 commit intosolana-labs:masterfrom
sakridge:shrink-debug

Conversation

@sakridge
Copy link
Copy Markdown
Contributor

Problem

Shrink message could be more informative and better formatted.

Summary of Changes

Add slot and fix <alive> / <stored to make sense with (forced) optional at the end.

Fixes #

@sakridge sakridge requested a review from ryoqun October 22, 2020 17:25
Copy link
Copy Markdown
Contributor

@ryoqun ryoqun left a comment

Choose a reason for hiding this comment

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

lgtm!

}
if alive_count == stored_count && stores.values().len() == 1 {
trace!(
"shrink_stale_slot: not able to shrink at all{}: {} / {}",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

err, last minute change at my #12110 broke this log... ;)

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 23, 2020

Codecov Report

Merging #13089 into master will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #13089   +/-   ##
=======================================
  Coverage    82.1%    82.1%           
=======================================
  Files         366      366           
  Lines       86335    86335           
=======================================
+ Hits        70955    70965   +10     
+ Misses      15380    15370   -10     

@ryoqun
Copy link
Copy Markdown
Contributor

ryoqun commented Oct 23, 2020

@ryoqun ryoqun added v1.3 v1.4 labels now

I quickly checked git log and it seems that this pr should be backported to v1.3 and v1.4 as well.

@sakridge sakridge merged commit 7d2729f into solana-labs:master Oct 23, 2020
@sakridge sakridge deleted the shrink-debug branch October 23, 2020 15:05
mergify Bot pushed a commit that referenced this pull request Oct 23, 2020
(cherry picked from commit 7d2729f)
mergify Bot pushed a commit that referenced this pull request Oct 23, 2020
(cherry picked from commit 7d2729f)
mergify Bot added a commit that referenced this pull request Oct 23, 2020
(cherry picked from commit 7d2729f)

Co-authored-by: sakridge <sakridge@gmail.com>
mergify Bot added a commit that referenced this pull request Oct 23, 2020
(cherry picked from commit 7d2729f)

Co-authored-by: sakridge <sakridge@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants