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
Solution Rebasing #545
base: master
Are you sure you want to change the base?
Solution Rebasing #545
Conversation
apps/arweave/src/ar_block_cache.erl
Outdated
%-type validation_status() :: on_chain | validated | not_validated | none. | ||
%-type nonce_limiter_validation_status() :: nonce_limiter_validated | | ||
% nonce_limiter_validation_scheduled | awaiting_nonce_limiter_validation. | ||
%-type block_status() :: validation_status() | {not_validated, nonce_limiter_validation_status()}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can just delete those if you want. I'd added them when I was trying to undesr the ar_block_cache for the checkpoint work. Probably not useful as a one-off
apps/arweave/src/ar_node_worker.erl
Outdated
@@ -1026,10 +740,10 @@ get_block_anchors_and_recent_txs_map(BlockTXPairs) -> | |||
apply_block(State) -> | |||
case ar_block_cache:get_earliest_not_validated_from_longest_chain(block_cache) of | |||
not_found -> | |||
%% Nothing to do - we are at the longest known chain already. | |||
{noreply, State}; | |||
may_be_rebase(State); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
may_be_rebase(State); | |
maybe_rebase(State); |
4fb89c0
to
c6ca466
Compare
@JamesPiechota FYI I pushed c6ca466 on top of previously reviewed. UPD nevermind, a similar change was in master already. |
c6ca466
to
c019a74
Compare
@ldmberman got it. let me know if you need me to re-review |
c019a74
to
0f08048
Compare
No description provided.