Question: how do you keep the lock

[spscommerce]

Question: how do you keep the lock

[xq5he]

@osallou Actually, set will return null if the key has been set. So this check will be Flase if the resource has been locked by other client.

[optimuspaul]

That does sound like a good feature request. My advice for now would be obtain locks with timeouts much longer than you expect you'd need.

I don't know if you'd read this, http://martin.kleppmann.com/2016/02/08/how-to-do-distributed-locking.html, but a lot of concerns about the safety of redlock have been brought up. I personally like the simplicity of redlock and in my use only use it for advice and not as a guarantee.

There is always the possibility that A fails to complete it's task before the timeout occurs, which is why you would never want to use redlock in a critical scenario unless you were also using a CAS of some sort to deal with conflict.

[osallou]

I indeed read article. A longer ttl will not fit for a process willing to
keep forever until stopped, released or crashed.
The random value obtained with the lock could be used certainly to allow an
expiration update only if value match, just like for releasing it.

Le ven. 18 mars 2016 18:52, Paul DeCoursey [email protected] a
écrit :

That does sound like a good feature request. My advice for now would be
obtain locks with timeouts much longer than you expect you'd need.

I don't know if you'd read this,
http://martin.kleppmann.com/2016/02/08/how-to-do-distributed-locking.html,
but a lot of concerns about the safety of redlock have been brought up. I
personally like the simplicity of redlock and in my use only use it for
advice and not as a guarantee.

There is always the possibility that A fails to complete it's task before
the timeout occurs, which is why you would never want to use redlock in a
critical scenario unless you were also using a CAS of some sort to deal
with conflict.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#17 (comment)