[IGNITE-17255] Implement ReplicaService

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Labels: ignite-3 transaction3_rw
• Epic Link: Ignite 3 transactions
• Assignee: Sergey Uttsel
• Reporter: Alexander Lapin
• Reviewer: Alexander Lapin
• Created: 28/Jun/22 19:49
• Updated: 30/Nov/22 14:31
• Resolved: 11/Aug/22 09:28

Description

For general context please check IGNITE-17252

Within given ticket it's required to

• Implement ReplicaService itself.
• Substitute RaftGroupService with ReplicaService from within InternalTableImpl and others

Please pay attention that according to tx protocol it's valid to fail the transaction in case of primary replica change - we'll introduce support of graceful primary replica switch later on. For now, within the scope of RW transactions it's enough to detect where primary replica is and enlist it to transaction with corresponding partition in order to reuse for further in-partition communication.

We should make it very clear, that any replicaService.invoke(nodeId) might fail with primaryReplicaMiss or replicaUnavailable, it's up to the outer logic to remap such failed requests.

However it's still required to detect proper primary replica initially and check whether it's still primary during further queries. Proper lease-based primary replica stability engine will be introduced within IGNITE-17256 , as a staring point it's possible to reuse sendWithRetry logic with true readIndex leader checks, meaning that primary replica is the replica collocated with the current leader (not the node that is thinking that it's a leader, but a node that was proved to be a leader based on readIndex logic).

In addition to all points mentioned above we should be aware that lot's of tests will become flaky  because sendWithRetry logic will only be available with initial primaryReplica detection method and not within common invokes. Generally speaking I believe that it's a good chance to rework them and thus make them stable.

Issue Links

• blocks: IGNITE-16882 Enlist txnState into single parition only
• is blocked by: IGNITE-16892 Introduce new LockManager interface in order to soupport S, X and I locks
• is blocked by: IGNITE-17252 Introduce Replica, ReplicaServer(?), ReplicaService and ReplicaListener interfaces
• is blocked by: IGNITE-17257 Implement Replica server-side logic
• is blocked by: IGNITE-17258 Implement ReplicaListener
• links to: GitHub Pull Request #996