[IGNITE-17759] Need to pass commitTableId and commitPartitionId to MvPartitionStorage#addWrite

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-beta1
• Labels: fix-version-validated ignite-3 transaction3_ro
• Epic Link: Ignite 3 transactions
• Ignite Flags: Docs Required, Release Notes Required
• Assignee: Vladislav Pyatkov
• Reporter: Sergey Uttsel
• Reviewer: Alexander Lapin
• Created: 26/Sep/22 09:37
• Updated: 01/Nov/22 11:10
• Resolved: 31/Oct/22 06:55

Description

Currently when PartitionListener invokes MvPartitionStorage#addWrite it passes 

UUID.randomUUID() as commitTableId and 0 as commitPartitionId. Need pass appropriate values.

 

For this:

1. Need to create

class PartitionId {
    UUID tableId;
    int partId;
}

1. In InternalTableImpl#enlistInTx need to save PartitionId of the first operation to the Transaction.
2. Need to change Map<String, IgniteBiTuple<ClusterNode, Long>> enlisted = new ConcurrentSkipListMap<>() to Map<PartitionId, IgniteBiTuple<ClusterNode, Long>> enlisted = new ConcurrentHashMap<>();
3. Need to change String groupId to PartitionId groupId in all places.
4. In InternalTableImpl#enlistInTx need to pass PartitionId into ReplicaRequest (ReadWriteSingleRowReplicaRequest, ReadWriteSwapRowReplicaRequest, ReadWriteMultiRowReplicaRequest)
5. In PartitionReplicaListener need to pass PartitionId from ReplicaRequest to UpdateCommand and UpdateAllCommand.

Issue Links

• is fixed by: IGNITE-17967 RO writeIntent resolution tests hang up in case of multi node cluster
• links to: PR