[IGNITE-6904] SQL: partition reservations are released too early in lazy mode

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.3
• Fix Version/s: 2.4
• Component/s: sql
• Assignee: Roman Kondakov
• Reporter: Vladimir Ozerov
• Created: 14/Nov/17 13:56
• Updated: 05/Apr/18 12:12
• Resolved: 25/Dec/17 09:41

Description

In lazy mode we advance query execution as new page requests arrive. However, method GridMapQueryExecutor#onQueryRequest0 releases partition reservations when only the very first page is processed:

finally {
    GridH2QueryContext.clearThreadLocal();

    if (distributedJoinMode == OFF)
        qctx.clearContext(false);
}

It means that incorrect results may be returned on unstable topology. We need to release partitions only after the whole query is executed.

Issue Links

• relates to: IGNITE-7712 Add an ability to globally enable 'lazy' flag for SQL queries
• links to: GitHub Pull Request #3074
• links to: TC run