-
Notifications
You must be signed in to change notification settings - Fork 892
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
Allow to configure BK for low latency busy-wait settings #1812
Conversation
do you have any perf numbers to share to show what kind of perf improvements we should expect with this feature enabled? |
@dlg99 I have posted the numbers here: #1641 (comment) |
bookkeeper-common/src/main/java/org/apache/bookkeeper/common/util/OrderedExecutor.java
Outdated
Show resolved
Hide resolved
bookkeeper-common/src/main/java/org/apache/bookkeeper/common/util/OrderedExecutor.java
Show resolved
Hide resolved
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Journal.java
Outdated
Show resolved
Hide resolved
@@ -602,8 +604,7 @@ BookieWatcher getBookieWatcher() { | |||
return bookieWatcher; | |||
} | |||
|
|||
@VisibleForTesting | |||
OrderedExecutor getMainWorkerPool() { | |||
public OrderedExecutor getMainWorkerPool() { |
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.
any reason why @VisibleForTesting
annotation is removed?
run java8 tests |
run integration tests |
rebuild java8 |
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.
the shading tests are failing because some jars are not correctly shaded.
run integration tests |
1 similar comment
run integration tests |
Motivation
Added
enableBusyWait
configuration option to turn on CPU Affinity and busy wait on sockets and queues.