-
Notifications
You must be signed in to change notification settings - Fork 860
prototype for decoupling execution from tendermint loop #2581
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
Conversation
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2581 +/- ##
==========================================
- Coverage 43.25% 43.25% -0.01%
==========================================
Files 1447 1447
Lines 115125 115123 -2
==========================================
- Hits 49799 49797 -2
Misses 61002 61002
Partials 4324 4324
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
| } | ||
|
|
||
| func (blockExec *BlockExecutor) handleFinalizeRequest(ctx context.Context, req *abci.RequestFinalizeBlock) error { | ||
| start := time.Now() |
Check warning
Code scanning / CodeQL
Calling the system time Warning
| } | ||
| // TODO: persist the corresponding ResponseCommit so we can reuse retain height after restarts. | ||
|
|
||
| saveBlockResponseTime := time.Now() |
Check warning
Code scanning / CodeQL
Calling the system time Warning
| blockExec.mempool.Lock() | ||
| defer blockExec.mempool.Unlock() | ||
|
|
||
| start := time.Now() |
Check warning
Code scanning / CodeQL
Calling the system time Warning
|
We truly appreciate your contribution and the time you’ve invested in this PR. |
|
This PR is being closed due to inactivity, but we want to acknowledge and thank you for the effort you’ve put in. |
Describe your changes and provide context
Testing performed to validate your change