-
Notifications
You must be signed in to change notification settings - Fork 780
Closed
Milestone
Description
Around 70% PBTS work was done on branch wb/proposer-based-timestamps (as sort of feature branch). However, that branch:
- wasn't merged back on
masterwith a merge commit - was rather "squash-merged" into a big PR: Proposer-Based Timestamps Merge tendermint/tendermint#7605.
This issue forward-ports tendermint/tendermint#7605 on our new feature branch feature/pbts.
- Forward port of tendermint/tendermint#7605, tendermint/spec#393, and tendermint/tendermint#8142 #2018
- Original PR: Proposer-Based Timestamps Merge tendermint/tendermint#7605, this is a big "merge-squash" PR.
- The original merge-squash PR contains the following smaller PRs (which have thus been forward-ported in this PR)
- state: add an 'IsTimely' method to implement the 'timely' check for proposer-based timestamps tendermint/tendermint#7170
- factory: simplify validator and genesis factory functions tendermint/tendermint#7305
- consensus: refactor the fake validator to take a clock source tendermint/tendermint#7300
- consensus: add calculation for proposal step waits from pbts tendermint/tendermint#7290
- consensus: ensure proposal receipt waits for maxWaitingTime tendermint/tendermint#7307
- types: remove accuracy from timestamp params tendermint/tendermint#7341
- internal/consensus: refactor ensure functions to use a common function tendermint/tendermint#7373
- types: add new consensus params from proto tendermint/tendermint#7354
- internal/consensus: proposer waits for previous block time tendermint/tendermint#7376
- Remove MedianTime, set block time to Now() tendermint/tendermint#7382
- internal/consensus: remove proposal wait time tendermint/tendermint#7418
- internal/consensus: prevote nil if proposal timestamp does not match tendermint/tendermint#7391
- Fix pbts tests tendermint/tendermint#7413
- consensus: check that proposal is non-nil before voting tendermint/tendermint#7480
- Prevote nil if proposal is not timely tendermint/tendermint#7415
- proto: rename timing params to synchrony params tendermint/tendermint#7554
- Remove votetime tendermint/tendermint#7563
- The original merge-squash PR contains the following smaller PRs (which have thus been forward-ported in this PR)
- This fw-port PR also includes two other PRs not included in Proposer-Based Timestamps Merge tendermint/tendermint#7605 but needed by it
- proto: merge the proposer-based timestamps parameters tendermint/spec#393 (protobufs, which were in a separate repo at the time)
- types: update synchrony params to match checked in proto tendermint/tendermint#8142
- Original PR: Proposer-Based Timestamps Merge tendermint/tendermint#7605, this is a big "merge-squash" PR.
DoD
- Forward-port Proposer-Based Timestamps Merge tendermint/tendermint#7605 and all other needed PRs
- Exclude unnedded refactoring not related to PBTS
- Production code builds
- Unit and e2e tests pass
Reactions are currently unavailable