Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Extract MAX_FINALITY_LAG constant from relay_chain_selection #5159

Merged
merged 5 commits into from
Mar 23, 2022

Conversation

tifecool
Copy link
Contributor

@tifecool tifecool commented Mar 18, 2022

PR attempts to solve #4788

Used MAX_HEADS_LOOK_BACK in approval voting as constant that MAX_BATCH_SCRAPE_ANCESTORS in dispute coordinator and MAX_FINALITY_LAG in relay chain selection are bound to.

polkadot address: 1s51dYXziQwTnSnModErKfra3TeEsVipqzY846CLCTEXtHJ

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Mar 18, 2022

User @tifecool, please sign the CLA here.

@tifecool
Copy link
Contributor Author

Moved constant to node primitive 👌

node/primitives/src/lib.rs Outdated Show resolved Hide resolved
@ordian
Copy link
Member

ordian commented Mar 18, 2022

cargo +nightly fmt is failing

tifecool and others added 2 commits March 18, 2022 18:11
Co-authored-by: Andronik <write@reusable.software>
@ordian ordian added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Mar 18, 2022
@tifecool tifecool requested a review from rphmeier March 21, 2022 16:36
@eskimor eskimor merged commit 3ffd40b into paritytech:master Mar 23, 2022
ordian added a commit that referenced this pull request Mar 24, 2022
* master:
  Try to fix out of view statements (#5177)
  Companion for Substrate#11107 (#5197)
  paras: `include_pvf_check_statement` rt bench (#4938)
  [ci] Run short benchmarks only in PR pipelines (#5200)
  Companion for paritytech/substrate#10242 (#4862)
  [ci] Add short benchmarks to the pipeline (#5188)
  upgrade coarsetime to 0.1.22 to fix a potential panic (#5193)
  Update docs and enable DOT-over-XCM (#4809)
  enable disputes on all chains (#5182)
  companion for validator self-vote in bags (#5088)
  Extract MAX_FINALITY_LAG constant from relay_chain_selection (#5159)
@tifecool tifecool deleted the fix4788 branch April 5, 2022 07:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants