Skip to content

Conversation

@clebertsuconic
Copy link
Contributor

@clebertsuconic clebertsuconic commented Jan 23, 2026

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.

@clebertsuconic clebertsuconic force-pushed the lock branch 18 times, most recently from 3e76798 to 940e9c4 Compare January 26, 2026 17:10
@clebertsuconic clebertsuconic force-pushed the lock branch 7 times, most recently from 3367529 to 8372c9c Compare January 27, 2026 01:48
@clebertsuconic
Copy link
Contributor Author

I just split this into 2 commits. they are still in the same PR as the etcd still depending on the first commit.

@clebertsuconic clebertsuconic force-pushed the lock branch 3 times, most recently from 521369c to cc8ee7d Compare January 27, 2026 02:05
@clebertsuconic clebertsuconic force-pushed the lock branch 6 times, most recently from b499bd5 to 5199de8 Compare January 28, 2026 15:07
@clebertsuconic
Copy link
Contributor Author

I think this is ready to be merged

@clebertsuconic clebertsuconic force-pushed the lock branch 5 times, most recently from b70f78d to 49ccc5b Compare January 28, 2026 20:57
@clebertsuconic clebertsuconic force-pushed the lock branch 3 times, most recently from 28c9429 to 2f8c62f Compare January 28, 2026 21:20
@jbertram
Copy link
Contributor

The commit message refers to both "Leader/Follower" and "LockMonitor." I might suggest something like this:

ARTEMIS-5852 Lock coordination for acceptors

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
Copy link
Contributor

jbertram commented Jan 28, 2026

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.

@clebertsuconic
Copy link
Contributor Author

@jbertram I am fine with adding a note.
I can add a note on the XSD and on the doc.

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.
@clebertsuconic
Copy link
Contributor Author

@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.

@clebertsuconic clebertsuconic changed the title ARTEMIS-5852 Leader/Follower pattern on acceptors ARTEMIS-5852 Lock coordination for acceptors Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants