[IGNITE-22861] Sql. Calculation for mapOnBackups parameter erased.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0-beta1
• Fix Version/s: 3.0
• Component/s: sql
• Labels: ignite-3
• Assignee: Andrey Mashenkov
• Reporter: Evgeny Stanilovsky
• Created: 30/Jul/24 09:33
• Updated: 04/Sep/24 16:08
• Resolved: 19/Aug/24 15:00

Description

After [1] was implemented it`s become possible to run queries on backup nodes but for now it erased, check implementation:

boolean mapOnBackups = tx.isReadOnly();
MappingParameters mappingParameters = MappingParameters.create(ctx.parameters(), mapOnBackups);

public static MappingParameters create(Object[] dynamicParameters, boolean mapOnBackups) {
        if (dynamicParameters.length == 0) {
            return EMPTY;
        } else {
            return new MappingParameters(dynamicParameters, mapOnBackups);
        }
    }

[1] IGNITE-22466 Sql. Support mapping to non-primary replicas

Issue Links

• Testing discovered: IGNITE-22969 Sql. Replanning query on unstable topology
• links to: GitHub Pull Request #4195