-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Labels
C:abciComponent: Application Blockchain InterfaceComponent: Application Blockchain InterfaceT:breakingType: Breaking ChangeType: Breaking Changebacklog-priorityA priority issue in the backlogA priority issue in the backlog
Description
Problem Definition
This was brought up in a similar issue. Currently, Tendermint uses a mutex around ABCI calls so that they are serialized. This was done out of safety and simplicity. However, this restricts the ability for apps to implement any form of concurrency between the applications. It should thus really be the responsibility of application to handle mutexes.
Proposal
Remove the mutex here:
tendermint/abci/client/local_client.go
Line 18 in e53f92b
| mtx *tmsync.RWMutex |
We should make sure that the change in responsibility is well documented
For Admin Use
- Not duplicate issue
- Appropriate labels applied
- Appropriate contributors tagged
- Contributor assigned/self-assigned
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C:abciComponent: Application Blockchain InterfaceComponent: Application Blockchain InterfaceT:breakingType: Breaking ChangeType: Breaking Changebacklog-priorityA priority issue in the backlogA priority issue in the backlog
Type
Projects
Status
Done/Merged