Using Dyno lock with conductor

[charybr]

Hi,

We have conductor using Dynomite for persistence and queuing. Conductor uses Dyno client to connect to Dynomite.
Now we want to enable locking in conductor for consistency. I had a look at this PR - #271.
Is there a way to enable locking using Dyno client? If yes, what are the configuration to be set on conductor side.

Best Regards,
Chary

[charybr]

I think we need to instantiate DynoLockClient in Conductor. Is it right?

[charybr]

Hi @rsrinivasanNetflix ,

dyno/dyno-recipes/src/main/java/com/netflix/dyno/recipes/lock/VotingHostsFromTokenRange.java

Line 36 in 0552980

effectiveVotingSize = votingSize == -1 ? MAX_VOTING_SIZE : votingSize;

In VotingHostsFromToken when votingSize is not provided (i.e DEFAULT_LOCK_VOTING_SIZE = -1 in ConnectionPoolConfigurationImpl) effectiveVotingSize is set to MAX_VOTING_SIZE (5). This makes it DynoLockClient quorum to 3. Is this intentional ? OR effectiveVotingSize should be set to MIN_VOTING_SIZE ?

effectiveVotingSize = votingSize == -1 ? MIN_VOTING_SIZE : votingSize;