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

Parallelize broadcast #1334

Merged
merged 1 commit into from
Jun 21, 2021
Merged

Conversation

longfin
Copy link
Member

@longfin longfin commented Jun 5, 2021

This PR parallelizes message broadcasting in Swarm<T>.

@longfin longfin added the network Related to networking (Libplanet.Net) label Jun 5, 2021
@longfin longfin self-assigned this Jun 5, 2021
@codecov
Copy link

codecov bot commented Jun 5, 2021

Codecov Report

Merging #1334 (a63626e) into main (eaabdcf) will increase coverage by 6.75%.
The diff coverage is 100.00%.

❗ Current head a63626e differs from pull request most recent head 1a8b758. Consider uploading reports for the commit 1a8b758 to get more accurate results

@@            Coverage Diff             @@
##             main    #1334      +/-   ##
==========================================
+ Coverage   76.89%   83.64%   +6.75%     
==========================================
  Files         253      410     +157     
  Lines       16984    36239   +19255     
==========================================
+ Hits        13059    30312   +17253     
- Misses       3398     4046     +648     
- Partials      527     1881    +1354     
Impacted Files Coverage Δ
Libplanet/Net/Transports/NetMQTransport.cs 79.22% <100.00%> (-2.26%) ⬇️
Libplanet/FixedSizedQueue.cs 81.25% <0.00%> (-18.75%) ⬇️
Libplanet/Store/BaseStore.cs 82.40% <0.00%> (-1.31%) ⬇️
Libplanet.RocksDBStore/RocksDBStore.cs 76.98% <0.00%> (-1.09%) ⬇️
Libplanet/Net/BlockCompletion.cs 94.20% <0.00%> (-0.87%) ⬇️
Libplanet.RocksDBStore/MonoRocksDBStore.cs 89.72% <0.00%> (-0.71%) ⬇️
Libplanet/Action/ActionEvaluation.cs 100.00% <0.00%> (ø)
Libplanet.Explorer/Store/MySQLRichStore.cs 0.00% <0.00%> (ø)
Libplanet.Explorer/Store/LiteDBRichStore.cs 0.00% <0.00%> (ø)
... and 166 more

@longfin longfin force-pushed the feature/parall-broadcast branch 2 times, most recently from 2f4e171 to eb22582 Compare June 5, 2021 12:52
@longfin longfin force-pushed the feature/parall-broadcast branch from eb22582 to 1a8b758 Compare June 18, 2021 17:32
@longfin longfin marked this pull request as ready for review June 18, 2021 17:34
@longfin
Copy link
Member Author

longfin commented Jun 21, 2021

@planetarium/libplanet PTAL

@longfin longfin merged commit 0b8da14 into planetarium:main Jun 21, 2021
longfin added a commit that referenced this pull request Jun 21, 2021
longfin added a commit that referenced this pull request Jun 21, 2021
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.

4 participants