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

Deterministic historian/accountant hashes #58

Merged
merged 1 commit into from
Mar 9, 2018

Conversation

garious
Copy link
Contributor

@garious garious commented Mar 9, 2018

When in tick-less mode, no longer continuously hash on the
background thread. That mode is just used for testing and
genesis log generation, and those extra hashes are just noise.

Note that without the extra hashes, with lose the duration between
events. Effectively, we distinguish proof-of-order from proof-of-time.

When in tick-less mode, no longer continuously hash on the
background thread. That mode is just used for testing and
genesis log generation, and those extra hashes are just noise.

Note that without the extra hashes, with lose the duration between
events. Effectively, we distinguish proof-of-order from proof-of-time.
@codecov-io
Copy link

Codecov Report

Merging #58 into master will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #58      +/-   ##
==========================================
+ Coverage   92.59%   92.63%   +0.04%     
==========================================
  Files          12       12              
  Lines         716      720       +4     
==========================================
+ Hits          663      667       +4     
  Misses         53       53
Impacted Files Coverage Δ
src/historian.rs 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d17aed...957fb06. Read the comment docs.

@garious garious merged commit 17926ff into solana-labs:master Mar 9, 2018
vkomenda pushed a commit to vkomenda/solana that referenced this pull request Aug 29, 2021
AshwinSekar pushed a commit to AshwinSekar/solana that referenced this pull request Mar 4, 2024
brooksprumo added a commit to brooksprumo/solana that referenced this pull request Mar 6, 2024
brooksprumo added a commit to brooksprumo/solana that referenced this pull request Mar 11, 2024
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.

2 participants