[IGNITE-19358] Sql. PrepareServiceImpl prepareDML cache does not take dynamic parameters into account

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0-alpha2
• Component/s: sql
• Labels: ignite-3
• Assignee: Unassigned
• Reporter: Maksim Zhuravkov
• Created: 25/Apr/23 09:35
• Updated: 15/Mar/24 08:31
• Resolved: 01/Sep/23 12:50

Description

PrepareService prepareDml cache does not take dynamic parameters into account which can potentially lead to runtime errors when different queries reuse the same plan.

    private CompletableFuture<QueryPlan> prepareDml(SqlNode sqlNode, PlanningContext ctx) {
        var key = new CacheKey(ctx.schemaName(), sqlNode.toString());
        ....

Expected behaviour: dml plan cache should take dynamic parameters/types of dynamic parameters into account.

Issue Links

• relates to: IGNITE-17765 Sql. Introduce cache for parsed statements