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

Remove non-Tendermint backends #1799

Closed
kostko opened this issue Jun 11, 2019 · 2 comments · Fixed by #1806
Closed

Remove non-Tendermint backends #1799

kostko opened this issue Jun 11, 2019 · 2 comments · Fixed by #1806
Assignees
Labels
c:registry Category: entity/node/runtime registry service c:roothash Category: root hash service

Comments

@kostko
Copy link
Member

kostko commented Jun 11, 2019

We should remove the following non-Tendermint backends as they have become a maintenance burden and are only used in some tests:

  • roothash,
  • registry,
  • epochtime,
  • committee scheduler,
  • beacon,
  • staking.

Going forward we should still maintain clear separation between a consensus backend (e.g., Tendermint) and our internal backends.

Requires #1748.

@kostko kostko added c:roothash Category: root hash service c:registry Category: entity/node/runtime registry service labels Jun 11, 2019
@Yawning
Copy link
Contributor

Yawning commented Jun 11, 2019

While we're at this, every Backend should be a BlockBackend where appropriate.

@kostko
Copy link
Member Author

kostko commented Jun 11, 2019

Yeah we should just get rid of BlockBackend and require height annotations in Backend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:registry Category: entity/node/runtime registry service c:roothash Category: root hash service
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants