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

Receipts #567

Merged
merged 26 commits into from
Nov 20, 2019
Merged

Receipts #567

merged 26 commits into from
Nov 20, 2019

Conversation

achamayou
Copy link
Member

@achamayou achamayou commented Nov 18, 2019

fix #56

Basic receipts support for CCF, subject to the compaction limit.

As a follow up to this, we should write the serialised tree in signature Txs, and sync up compaction with commit, to allow historical verification.

@ghost
Copy link

ghost commented Nov 18, 2019

images

@codecov-io
Copy link

codecov-io commented Nov 18, 2019

Codecov Report

Merging #567 into master will decrease coverage by 0.09%.
The diff coverage is 69.62%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
- Coverage   78.56%   78.47%   -0.09%     
==========================================
  Files         140      140              
  Lines       10567    10646      +79     
==========================================
+ Hits         8301     8354      +53     
- Misses       2266     2292      +26
Flag Coverage Δ
#e2e_BFT 50.75% <8.86%> (-0.16%) ⬇️
#e2e_CFT 73.37% <69.62%> (-0.03%) ⬇️
#unit_BFT 64.84% <0%> (-0.62%) ⬇️
#unit_CFT 72.03% <8.86%> (-0.73%) ⬇️
Impacted Files Coverage Δ
src/kv/kvtypes.h 60% <ø> (ø) ⬆️
src/node/rpc/frontend.h 75.59% <52.5%> (-2.16%) ⬇️
src/node/history.h 61.62% <87.18%> (+6.27%) ⬆️
src/host/tcp.h 73.08% <0%> (-0.85%) ⬇️

src/node/rpc/frontend.h Outdated Show resolved Hide resolved
src/node/history.h Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Nov 19, 2019

images

src/node/history.h Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Nov 19, 2019

images

@ghost
Copy link

ghost commented Nov 19, 2019

images

@achamayou achamayou marked this pull request as ready for review November 19, 2019 13:57
@achamayou achamayou requested a review from a team as a code owner November 19, 2019 13:57
src/node/rpc/frontend.h Outdated Show resolved Hide resolved
@jumaffre
Copy link
Contributor

Could this new feature be (briefly) documented here: https://microsoft.github.io/CCF/users/issue_commands.html#checking-for-commit?

@achamayou
Copy link
Member Author

@jumaffre sure, I'll make it short for now because I'm planning to do a single txInfo RPC as discussed.

@ghost
Copy link

ghost commented Nov 19, 2019

images

@ghost
Copy link

ghost commented Nov 19, 2019

images

@ghost
Copy link

ghost commented Nov 20, 2019

images

@ghost
Copy link

ghost commented Nov 20, 2019

images

@ghost
Copy link

ghost commented Nov 20, 2019

images

@achamayou achamayou merged commit 0b743c4 into master Nov 20, 2019
@achamayou achamayou deleted the receipts branch November 20, 2019 16:38
eddyashton pushed a commit to eddyashton/CCF that referenced this pull request Mar 24, 2020
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.

Optional transaction receipts
6 participants