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

Broadcast single block header instead of block hashes #764

Merged
merged 5 commits into from
Jan 22, 2020

Conversation

limebell
Copy link
Member

We can ignore a block in different branch with lower index by this patch. See #757.

@limebell limebell added the network Related to networking (Libplanet.Net) label Jan 16, 2020
@limebell limebell self-assigned this Jan 16, 2020
@limebell limebell force-pushed the refactor/header-based-message branch 2 times, most recently from 386b14e to fde68f6 Compare January 16, 2020 05:53
earlbread
earlbread previously approved these changes Jan 16, 2020
CHANGES.md Outdated Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
Libplanet/Net/Swarm.cs Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
Libplanet/Net/Swarm.cs Outdated Show resolved Hide resolved
Libplanet/Net/Messages/BlockHeaderMessage.cs Outdated Show resolved Hide resolved
@moreal
Copy link
Contributor

moreal commented Jan 16, 2020

This seems to need to resolve the conflict and to be rebased on the current master.

dahlia
dahlia previously approved these changes Jan 16, 2020
moreal
moreal previously approved these changes Jan 16, 2020
earlbread
earlbread previously approved these changes Jan 17, 2020
Libplanet/Net/Swarm.cs Outdated Show resolved Hide resolved
Libplanet/Net/Swarm.cs Outdated Show resolved Hide resolved
@limebell limebell requested a review from dahlia January 21, 2020 06:28
dahlia
dahlia previously approved these changes Jan 21, 2020
longfin
longfin previously approved these changes Jan 22, 2020
@limebell limebell dismissed stale reviews from longfin and dahlia via 1331417 January 22, 2020 06:03
@limebell limebell force-pushed the refactor/header-based-message branch from 3f8fd77 to 1331417 Compare January 22, 2020 06:03
@codecov
Copy link

codecov bot commented Jan 22, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@faa07d5). Click here to learn what that means.
The diff coverage is 80.68%.

@@            Coverage Diff            @@
##             master     #764   +/-   ##
=========================================
  Coverage          ?   86.32%           
=========================================
  Files             ?      225           
  Lines             ?    19280           
  Branches          ?        0           
=========================================
  Hits              ?    16644           
  Misses            ?     1434           
  Partials          ?     1202
Impacted Files Coverage Δ
Libplanet/Net/Messages/BlockHashes.cs 100% <100%> (ø)
Libplanet/Net/Messages/BlockHeaderMessage.cs 100% <100%> (ø)
Libplanet/Net/Messages/Message.cs 85.71% <100%> (ø)
Libplanet/Blocks/BlockHeader.cs 85.71% <58.33%> (ø)
Libplanet/Net/Swarm.cs 84.29% <72.5%> (ø)
Libplanet.Tests/Net/SwarmTest.cs 96.32% <95.23%> (ø)

@moreal moreal requested review from dahlia and longfin January 22, 2020 06:07
@limebell limebell merged commit 5bcc16e into planetarium:master Jan 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
network Related to networking (Libplanet.Net)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants