-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat(abciclient): route abci requests depending on request type #734
Conversation
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
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.
👍
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.
👍
Issue being fixed or feature implemented
We want to have separate ABCI app instance for read-only operations, like check tx or query, and separate for
consensus operations like prepare/process proposal, finalize block, etc.
What was done?
Implemented RoutedClient that routes requests to one of multiple abci apps.
It can be configured by settings:
Documented in config.toml.
How Has This Been Tested?
Integrated into Dash Platform's rs-drive-abci
Breaking Changes
None
Checklist:
For repository code-owners and collaborators only