-
Notifications
You must be signed in to change notification settings - Fork 13
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
adding an option for outbound concurrency control #36
adding an option for outbound concurrency control #36
Conversation
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Codecov Report
@@ Coverage Diff @@
## main #36 +/- ##
==========================================
- Coverage 98.80% 98.39% -0.41%
==========================================
Files 34 34
Lines 2674 2685 +11
==========================================
Hits 2642 2642
- Misses 25 35 +10
- Partials 7 8 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Think this is a great addition @Chengxuan
One request to avoid blocking infinitely, rather than up to a context timeout
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Peter Broadhurst <peter.broadhurst@kaleido.io>
This will drop coverage I think, so I'll submit a follow-on PR to address that @Chengxuan |
Some blockchain endpoint has a concurrent limit, add an option here to control the number of concurrent requests.