vldpyatkov commented on code in PR #13264:
URL: https://github.com/apache/ignite/pull/13264#discussion_r3481340405
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/colocated/GridDhtColocatedCache.java:
##########
@@ -1077,7 +1086,7 @@ private IgniteInternalFuture<Exception> lockAllAsync0(
@Override public Exception apply(Boolean b, Exception e) {
if (e != null)
e = U.unwrap(e);
- else if (!b)
+ else if (!b && !waitTimeoutExpiresFirst(waitTimeout,
timeout))
Review Comment:
If the nested future result (`fut` actually) finished with false (it means
the lock was not owned) and we are in the lockTxEntry method (because
`waitTimeoutExpiresFirst`) completes the embedded future with
GridCacheLockTimeoutException (as a result, because the future result is an
exception).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
