Hey Dmitry,
sorry for the late reply, I'll try to bake a pr later during the day.
Best regards,
Vladisav
On Tue, Apr 4, 2017 at 11:05 AM, Dmitry Karachentsev
<dkarachent...@gridgain.com <mailto:dkarachent...@gridgain.com>>
wrote:
Hi Vladislav,
I see you're developing [1] for a while, did you have any
chance to fix it? If no, is there any estimate?
[1] https://issues.apache.org/jira/browse/IGNITE-1977
<https://issues.apache.org/jira/browse/IGNITE-1977>
Thanks!
-Dmitry.
20.03.2017 10:28, Alexey Goncharuk пишет:
I think re-creation should be handled by a user who will
make sure that
nobody else is currently executing the guarded logic
before the
re-creation. This is exactly the same semantics as with
BrokenBarrierException for j.u.c.CyclicBarrier.
2017-03-17 2:39 GMT+03:00 Vladisav Jelisavcic
<vladis...@gmail.com <mailto:vladis...@gmail.com>>:
Hi everyone,
I agree with Val, he's got a point; recreating the
lock doesn't seem
possible
(at least not the with the transactional cache
lock/semaphore we have).
Is this re-create behavior really needed?
Best regards,
Vladisav
On Thu, Mar 16, 2017 at 8:34 PM, Valentin Kulichenko <
valentin.kuliche...@gmail.com
<mailto:valentin.kuliche...@gmail.com>> wrote:
Guys,
How does recreation of the lock helps? My
understanding is that scenario
is
the following:
1. Client A creates and acquires a lock, and then
starts to execute
guarded
logic.
2. Client B tries to acquire the same lock and
parks to wait.
3. Before client A unlocks, all affinity nodes
for the lock fail, lock
disappears from the cache.
4. Client B fails with exception, recreates the
lock, acquires it, and
starts to execute guarded logic concurrently with
client A.
In my view this is wrong anyway, regardless of
whether this happens
silently or with an exception handled in user's
code. Because this code
doesn't have any way to know if client A still
holds the lock or not.
Am I missing something?
-Val
On Tue, Mar 14, 2017 at 10:14 AM, Dmitriy Setrakyan <
dsetrak...@apache.org <mailto:dsetrak...@apache.org>
wrote:
On Tue, Mar 14, 2017 at 12:46 AM, Alexey
Goncharuk <
alexey.goncha...@gmail.com
<mailto:alexey.goncha...@gmail.com>> wrote:
Which user operation would result in
exception? To my knowledge,
user
may
already be holding the lock and not
invoking any Ignite APIs, no?
Yes, this is exactly my point.
Imagine that a node already holds a lock
and another node is waiting
for
the lock. If all partition nodes leave
the grid and the lock is
re-created,
this second node will immediately acquire
the lock and we will have
two
lock owners. I think in this case this
second node (blocked on
lock())
should get an exception saying that the
lock was lost (which is, by
the
way, the current behavior), and the first
node should get an
exception
on
unlock.
Makes sense.