-
Notifications
You must be signed in to change notification settings - Fork 504
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
oltp concurrent requests share multi executors instead of one executor #1258
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1258 +/- ##
============================================
- Coverage 69.19% 62.33% -6.86%
- Complexity 5401 5767 +366
============================================
Files 328 374 +46
Lines 26334 31355 +5021
Branches 3750 4401 +651
============================================
+ Hits 18222 19546 +1324
- Misses 6333 9826 +3493
- Partials 1779 1983 +204
Continue to review full report at Codecov.
|
return; | ||
} | ||
int workers = this.config().get(CoreOptions.OLTP_CONCURRENT_THREADS); | ||
if (workers > 0) { | ||
executor = Consumers.newThreadPool(EXECUTOR_NAME, workers); | ||
executors = new Consumers.ExecutorPool(EXECUTOR_NAME, workers); |
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.
If create Consumers.ExecutorPool for every OltpTraverser, how share the executors in Consumers.ExecutorPool?
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.
all OltpTraverser share one ExecutorPool (executors)
also reduce the wake-up period of consumers from 1s to 1ms Change-Id: I2631e03cd4c60fce5a337c138f6921a59beaa24d
471a813
to
55482ea
Compare
apache#1258) also reduce the wake-up period of consumers from 1s to 1ms Change-Id: I2631e03cd4c60fce5a337c138f6921a59beaa24d
also reduce the wake-up period of consumers from 1s to 1ms
Change-Id: I2631e03cd4c60fce5a337c138f6921a59beaa24d