[IGNITE-21375] RO transactions should not update txnState local map

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0
• Labels: ignite-3 performance
• Epic Link: performance enhancement
• Assignee: Vladislav Pyatkov
• Reporter: Alexander Lapin
• Reviewer: Kirill Sizov
• Created: 29/Jan/24 11:40
• Updated: 04/Sep/24 16:07
• Resolved: 09/Feb/24 09:42

Description

Motivation

On the one hand according to vpyatkov updating txnstate within RO flow brings us 20% performance drop, on the other hand RO transaction updates corresponding state only for testing purposes, precisely in order to check whether transaction rollback was called. See org.apache.ignite.internal.tx.TxManager#pending for more details.

Definition of Done

• Eliminate txnState local map updates within RO txn flow.
• Implement different mechanism for txn cleanup verification instead of txnState local map based one.

Issue Links

• is related to: IGNITE-21378 Investigate whether it's possible to make txnState local map updates faster
• links to: GitHub Pull Request #3145