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

Break when getting blocks count is less then hash chunk size #1112

Merged
merged 1 commit into from
Nov 26, 2020

Conversation

riemannulus
Copy link
Member

If getting blocks size in Swarm<T>.FillBlocksAsync() is less then Swarm<T>.FindNextHashesChunkSize, it means getting all of blocks in this try. So, do not processing next step for check.

@riemannulus riemannulus self-assigned this Nov 26, 2020
Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Could we have a small unit test for this? (If it's not easy we should have a TODO/FIXME comment then.)
  • This behavioral change should be recorded in the changelog.

Libplanet/Net/Swarm.cs Outdated Show resolved Hide resolved
Libplanet/Net/Swarm.cs Outdated Show resolved Hide resolved
Libplanet/Net/Swarm.cs Outdated Show resolved Hide resolved
@riemannulus riemannulus force-pushed the fix/fillblock branch 2 times, most recently from 09fb3be to 969588c Compare November 26, 2020 07:53
longfin
longfin previously approved these changes Nov 26, 2020
longfin
longfin previously approved these changes Nov 26, 2020
CHANGES.md Outdated Show resolved Hide resolved
dahlia
dahlia previously approved these changes Nov 26, 2020
@dahlia dahlia requested a review from longfin November 26, 2020 07:58
longfin
longfin previously approved these changes Nov 26, 2020
@longfin longfin merged commit 956295a into planetarium:main Nov 26, 2020
riemannulus pushed a commit that referenced this pull request Nov 30, 2020
Test for break when getting blocks count is less then hash chunk size #1112
@riemannulus riemannulus deleted the fix/fillblock branch January 27, 2023 09:26
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.

4 participants