[IGNITE-16290] Non-informative warning on query retry - Query run was already removed

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.13
• Component/s: sql
• Labels: ise.lts
• Assignee: Andrey Mashenkov
• Reporter: Andrey Mashenkov
• Created: 13/Jan/22 10:17
• Updated: 13/May/22 10:13
• Resolved: 14/Jan/22 12:01

Description

The non-informative warning is printed when SQL query is restarted.

[WARN ][pub-#3460][GridReduceQueryExecutor] Query run was already removed: 1467
2[2021-11-28T05:30:25,813][WARN ][pub-#3460][GridReduceQueryExecutor] Query run was already removed: 1468
3[2021-11-28T05:30:25,874][WARN ][pub-#3460][GridReduceQueryExecutor] Query run was already removed: 1469

Theis happens because we remove query context (ReduceQueryRun) from a map twice: when decide to retry the query 

                    if (retry) {
                        lastRun = runs.remove(qryReqId);
                        assert lastRun != null;
                    } else { 

then when releasing resources in a finally block.

finally {
    if (release) {
        releaseRemoteResources(nodes, r, qryReqId, qry.distributedJoins(), mvccTracker); 

Thus, the second try is failed with the log message.

Issue Links

• links to: GitHub Pull Request #9733