-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Include shred version in gossip #7777
Conversation
sagar-solana
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too much broke when you added the filter logic?
Yeah, 🤕, so I scaled this back a little. Still trying to figure out what |
Pull request has been modified.
|
|
|
|
||
| let mut i = 0; | ||
| while value.size() < desired_size { | ||
| while value.size() <= desired_size { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<= instead of < got test_split_messages_packet_size() working again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah okay, so the extra 2 bytes made it match exactly. I think that should be fine.
Codecov Report
@@ Coverage Diff @@
## master #7777 +/- ##
========================================
- Coverage 81.8% 81.8% -0.1%
========================================
Files 241 241
Lines 51075 51023 -52
========================================
- Hits 41817 41759 -58
- Misses 9258 9264 +6 |
Nodes with differing shred versions will not ingest each other's shreds, so ignore tvu/rpc nodes on gossip with differing shred versions when
--wait-for-supermajorityvalidator argument is used.