-
Notifications
You must be signed in to change notification settings - Fork 948
ARTEMIS-5852 Lock coordination for acceptors #6191
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
base: main
Are you sure you want to change the base?
Conversation
3e76798 to
940e9c4
Compare
...ain/java/org/apache/activemq/artemis/lockmanager/etcd/EtcdDistributedLockManagerFactory.java
Outdated
Show resolved
Hide resolved
.../src/main/java/org/apache/activemq/artemis/lockmanager/file/FileBasedLockManagerFactory.java
Outdated
Show resolved
Hide resolved
.../org/apache/activemq/artemis/lockmanager/zookeeper/CuratorDistributedLockManagerFactory.java
Outdated
Show resolved
Hide resolved
3367529 to
8372c9c
Compare
|
I just split this into 2 commits. they are still in the same PR as the etcd still depending on the first commit. |
521369c to
cc8ee7d
Compare
b499bd5 to
5199de8
Compare
|
I think this is ready to be merged |
b70f78d to
49ccc5b
Compare
...mis-server/src/main/java/org/apache/activemq/artemis/core/config/impl/ConfigurationImpl.java
Outdated
Show resolved
Hide resolved
28c9429 to
2f8c62f
Compare
|
The commit message refers to both "Leader/Follower" and "LockMonitor." I might suggest something like this: |
|
Would it make sense to mark this as tech preview for the time being just so we're not 100% locked into the config, API, etc.? I don't have a strong feeling either way, but thought I'd mention it. FWIW, we did that before with the locking stuff that Franz implemented. |
|
@jbertram I am fine with adding a note. Do you know of anywhere else I should add this? |
I'm adding a LockCoordinator to the broker, that will use DistributedLock to help start and stop acceptors. You can associate the LockCoordinator with acceptors and an acceptor serving only clients would then be activated in only one of the brokers.
|
@jbertram I added the tech preview note. this note actually gives me a piece of mind merging this... giving me an opportunity to eventually change it. thanks for this suggestion. |
I'm adding a LockCoordinator to the broker, that will use DistributedLock
to help start and stop acceptors.
You can associate the LockCoordinator with acceptors and an acceptor
serving only clients would then be activated in only one of the brokers.