denis-chudov commented on code in PR #2918:
URL: https://github.com/apache/ignite-3/pull/2918#discussion_r1422885146
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/replicator/TransactionStateResolver.java:
##########
@@ -218,18 +222,17 @@ private void resolveTxStateFromTxCoordinator(
if (coordinator == null) {
// This means the coordinator node have either left the cluster or
restarted.
+ markAbandoned(txId);
Review Comment:
We will, but we still should put the tx in this state because it's not
correct not to do it. For example:
- RO transaction tx0 resolves write intent belonging to the transaction tx1
and marks it as abandoned and starts the recovery;
- after that RW transaction tx2 meets the lock belonging to tx1, sees that
it's abandoned recently and doesn't start the recovery, minimizing redundant
network messages.
--
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]
