Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

fix: discard intermediate proofs #106

Merged
merged 10 commits into from
Jun 14, 2024
Merged

Conversation

atanmarko
Copy link
Member

Resolves #104

@atanmarko atanmarko self-assigned this Jun 11, 2024
@atanmarko atanmarko marked this pull request as ready for review June 11, 2024 15:08
leader/src/jerigon.rs Outdated Show resolved Hide resolved
leader/src/jerigon.rs Outdated Show resolved Hide resolved
leader/src/jerigon.rs Outdated Show resolved Hide resolved
@atanmarko atanmarko changed the title fix: discard intermediary proofs WIP: fix: discard intermediary proofs Jun 11, 2024
@atanmarko atanmarko marked this pull request as draft June 11, 2024 16:29
@atanmarko atanmarko changed the title WIP: fix: discard intermediary proofs WIP: fix: discard intermediate proofs Jun 11, 2024
@atanmarko atanmarko marked this pull request as ready for review June 12, 2024 12:39
@atanmarko atanmarko changed the title WIP: fix: discard intermediate proofs fix: discard intermediate proofs Jun 12, 2024
@atanmarko atanmarko requested a review from Nashtare June 12, 2024 12:39
@Nashtare Nashtare added this to the Cleanups and Misc. milestone Jun 12, 2024
tools/prove_blocks.sh Outdated Show resolved Hide resolved
leader/src/jerigon.rs Outdated Show resolved Hide resolved
leader/src/jerigon.rs Outdated Show resolved Hide resolved
leader/src/jerigon.rs Outdated Show resolved Hide resolved
@atanmarko atanmarko force-pushed the fix/discard-intermediary-proofs branch from e68d5aa to 4f369ff Compare June 14, 2024 10:53
@atanmarko atanmarko force-pushed the fix/discard-intermediary-proofs branch from 4f369ff to bf05f9c Compare June 14, 2024 10:54
@atanmarko atanmarko force-pushed the fix/discard-intermediary-proofs branch from b5de6a6 to 39ece21 Compare June 14, 2024 10:59
@atanmarko atanmarko requested a review from Nashtare June 14, 2024 10:59
Copy link
Collaborator

@Nashtare Nashtare left a comment

Choose a reason for hiding this comment

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

LGTM. I can't test it as I don't have any Jerigon node endpoint atm, but I trust you did :)

leader/src/jerigon.rs Show resolved Hide resolved
@atanmarko atanmarko merged commit b6bc8c8 into develop Jun 14, 2024
4 checks passed
@atanmarko atanmarko deleted the fix/discard-intermediary-proofs branch June 14, 2024 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Do not persist all intermediary block proofs to disk
2 participants