Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More SlotMeta docs #3011

Merged
merged 3 commits into from
Feb 28, 2019
Merged

More SlotMeta docs #3011

merged 3 commits into from
Feb 28, 2019

Conversation

garious
Copy link
Contributor

@garious garious commented Feb 28, 2019

Problem

SlotMeta descriptions lack precision

Summary of Changes

Take a stab at creating at better ones

@garious garious requested a review from rob-solana February 28, 2019 18:04
@codecov
Copy link

codecov bot commented Feb 28, 2019

Codecov Report

Merging #3011 into master will increase coverage by 2.7%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #3011     +/-   ##
========================================
+ Coverage      74%   76.8%   +2.7%     
========================================
  Files         129     129             
  Lines       20519   19786    -733     
========================================
- Hits        15199   15197      -2     
+ Misses       5320    4589    -731

src/blocktree.rs Outdated Show resolved Hide resolved
src/blocktree.rs Outdated Show resolved Hide resolved
@garious garious merged commit fec8675 into solana-labs:master Feb 28, 2019
yihau pushed a commit to yihau/solana that referenced this pull request Oct 2, 2024
…olana-labs#3006) (solana-labs#3011)

removes early return if prune_messages are empty (solana-labs#3006)

Even if there are no outgoing prune messages we still need to generate
outgoing push messages for packets just received, so the code should not
early return here:
https://github.com/anza-xyz/agave/blob/d2cc71f0d/gossip/src/cluster_info.rs#L2400-L2402

(cherry picked from commit ce15821)

Co-authored-by: behzad nouri <[email protected]>
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