[ 
https://issues.apache.org/jira/browse/IGNITE-28957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18105302#comment-18105302
 ] 

Ignite TC Bot commented on IGNITE-28957:
----------------------------------------

{panel:title=Branch: [pull/13482/head] Base: [master] : No blockers 
found!|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}{panel}
{panel:title=Branch: [pull/13482/head] Base: [master] : New Tests 
(1)|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}
{color:#00008b}Calcite SQL 3{color} [[tests 
1|https://ci2.ignite.apache.org/viewLog.html?buildId=9289441]]
* {color:#013220}IgniteCalciteTestSuite3: 
MemoryQuotasIntegrationTest.testSelectForUpdateExceedsMemoryQuota - 
PASSED{color}

{panel}
[TeamCity *--> Run :: All* 
Results|https://ci2.ignite.apache.org/viewLog.html?buildId=9289474&buildTypeId=IgniteTests24Java8_RunAll]
{color:#ffffff}tcbot-analysis-comment chainBuildId=9289474 
rerunBuildIds=9289782,9289784{color}

> Calcite SELECT FOR UPDATE may cause OOM by materializing the entire result set
> ------------------------------------------------------------------------------
>
>                 Key: IGNITE-28957
>                 URL: https://issues.apache.org/jira/browse/IGNITE-28957
>             Project: Ignite
>          Issue Type: Bug
>          Components: sql
>            Reporter: Vladislav Pyatkov
>            Assignee: Vladislav Pyatkov
>            Priority: Major
>              Labels: calcite, sql
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> h2. Motivation
> `ExecutionServiceImpl#tryExecuteForUpdate` calls 
> `ListFieldsQueryCursor#getAll()` for the inner SELECT before acquiring row 
> locks:
> {code}
> ListFieldsQueryCursor<?> innerCursor = mapAndExecutePlan(qry, 
> plan.innerPlan());
> List<List<?>> rows = innerCursor.getAll();
> {code}
> As a result, the complete query result is materialized on the coordinator.
> A broad SELECT FOR UPDATE, especially one involving large values or joins, 
> may consume an unbounded amount of heap and eventually cause an 
> OutOfMemoryError.
> h2. Definition of done
> The implementation should avoid keeping the entire result set in memory.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to