-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Conversation
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.
|
@@ -2589,7 +2590,7 @@ mod tests { | |||
})); | |||
|
|||
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-supermajority
validator argument is used.