[jira] [Updated] (IGNITE-13136) Calcite integration. Improve join predicate testing.

2021-11-16 Thread Taras Ledkov (Jira) 


 [ 
https://issues.apache.org/jira/browse/IGNITE-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Taras Ledkov updated IGNITE-13136:
--
Labels: calcite ignite-3  (was: calcite calcite3-required ignite-3)

> Calcite integration. Improve join predicate testing.
> 
>
> Key: IGNITE-13136
> URL: https://issues.apache.org/jira/browse/IGNITE-13136
> Project: Ignite
>  Issue Type: Improvement
>  Components: sql
>Reporter: Roman Kondakov
>Assignee: Evgeny Stanilovsky
>Priority: Minor
>  Labels: calcite, ignite-3
>  Time Spent: 4.5h
>  Remaining Estimate: 0h
>
> Currently we have to merge joining rows in order to test a join predicate:
> {code:java}
> Row row = handler.concat(left, rightMaterialized.get(rightIdx++));
> if (!cond.test(row))
> continue;
> {code}
> it results in unconditional building a joined row even if it will not be 
> emitted to downstream further. To avoid extra GC pressure we need to test the 
> join predicate before joining rows:
> {code:java}
> if (!cond.test(left, right))
> continue;
> Row row = handler.concat(left, right);
> {code}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

[jira] [Updated] (IGNITE-13136) Calcite integration. Improve join predicate testing.

2021-10-08 Thread Evgeny Stanilovsky (Jira) 


 [ 
https://issues.apache.org/jira/browse/IGNITE-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Evgeny Stanilovsky updated IGNITE-13136:

Labels: calcite calcite3-required ignite-3  (was: calcite calcite3-required)

> Calcite integration. Improve join predicate testing.
> 
>
> Key: IGNITE-13136
> URL: https://issues.apache.org/jira/browse/IGNITE-13136
> Project: Ignite
>  Issue Type: Improvement
>  Components: sql
>Reporter: Roman Kondakov
>Assignee: Evgeny Stanilovsky
>Priority: Minor
>  Labels: calcite, calcite3-required, ignite-3
>  Time Spent: 4.5h
>  Remaining Estimate: 0h
>
> Currently we have to merge joining rows in order to test a join predicate:
> {code:java}
> Row row = handler.concat(left, rightMaterialized.get(rightIdx++));
> if (!cond.test(row))
> continue;
> {code}
> it results in unconditional building a joined row even if it will not be 
> emitted to downstream further. To avoid extra GC pressure we need to test the 
> join predicate before joining rows:
> {code:java}
> if (!cond.test(left, right))
> continue;
> Row row = handler.concat(left, right);
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

[jira] [Updated] (IGNITE-13136) Calcite integration. Improve join predicate testing.

2021-07-30 Thread Evgeny Stanilovsky (Jira) 


 [ 
https://issues.apache.org/jira/browse/IGNITE-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Evgeny Stanilovsky updated IGNITE-13136:

Labels: calcite calcite3-required  (was: calcite calcite2-required 
calcite3-required)

> Calcite integration. Improve join predicate testing.
> 
>
> Key: IGNITE-13136
> URL: https://issues.apache.org/jira/browse/IGNITE-13136
> Project: Ignite
>  Issue Type: Improvement
>  Components: sql
>Reporter: Roman Kondakov
>Assignee: Evgeny Stanilovsky
>Priority: Minor
>  Labels: calcite, calcite3-required
>  Time Spent: 4h 10m
>  Remaining Estimate: 0h
>
> Currently we have to merge joining rows in order to test a join predicate:
> {code:java}
> Row row = handler.concat(left, rightMaterialized.get(rightIdx++));
> if (!cond.test(row))
> continue;
> {code}
> it results in unconditional building a joined row even if it will not be 
> emitted to downstream further. To avoid extra GC pressure we need to test the 
> join predicate before joining rows:
> {code:java}
> if (!cond.test(left, right))
> continue;
> Row row = handler.concat(left, right);
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

[jira] [Updated] (IGNITE-13136) Calcite integration. Improve join predicate testing.

2021-06-18 Thread Yury Gerzhedovich (Jira) 


 [ 
https://issues.apache.org/jira/browse/IGNITE-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yury Gerzhedovich updated IGNITE-13136:
---
Labels: calcite2-required calcite3-required  (was: )

> Calcite integration. Improve join predicate testing.
> 
>
> Key: IGNITE-13136
> URL: https://issues.apache.org/jira/browse/IGNITE-13136
> Project: Ignite
>  Issue Type: Improvement
>  Components: sql
>Reporter: Roman Kondakov
>Priority: Minor
>  Labels: calcite2-required, calcite3-required
>
> Currently we have to merge joining rows in order to test a join predicate:
> {code:java}
> Row row = handler.concat(left, rightMaterialized.get(rightIdx++));
> if (!cond.test(row))
> continue;
> {code}
> it results in unconditional building a joined row even if it will not be 
> emitted to downstream further. To avoid extra GC pressure we need to test the 
> join predicate before joining rows:
> {code:java}
> if (!cond.test(left, right))
> continue;
> Row row = handler.concat(left, right);
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

[jira] [Updated] (IGNITE-13136) Calcite integration. Improve join predicate testing.

2020-06-09 Thread Roman Kondakov (Jira) 


 [ 
https://issues.apache.org/jira/browse/IGNITE-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Roman Kondakov updated IGNITE-13136:

Description: 
Currently we have to merge joining rows in order to test a join predicate:
{code:java}
Row row = handler.concat(left, rightMaterialized.get(rightIdx++));

if (!cond.test(row))
continue;
{code}
it results in unconditional building a joined row even if it will not be 
emitted to downstream further. To avoid extra GC pressure we need to test the 
join predicate before joining rows:
{code:java}
if (!cond.test(left, right))
continue;

Row row = handler.concat(left, right);
{code}

  was:
Currently we have to merge joining rows in order to test a join predicate:
{code:java}
Row row = handler.concat(left, rightMaterialized.get(rightIdx++));

if (!cond.test(row))
continue;
{code}
it results in unconditional building a joining row even if it will not be 
emitted to downstream further. To avoid extra GC pressure we need to test the 
join predicate before joining rows:
{code:java}
if (!cond.test(left, right))
continue;

Row row = handler.concat(left, right);
{code}


> Calcite integration. Improve join predicate testing.
> 
>
> Key: IGNITE-13136
> URL: https://issues.apache.org/jira/browse/IGNITE-13136
> Project: Ignite
>  Issue Type: Improvement
>  Components: sql
>Reporter: Roman Kondakov
>Priority: Minor
>
> Currently we have to merge joining rows in order to test a join predicate:
> {code:java}
> Row row = handler.concat(left, rightMaterialized.get(rightIdx++));
> if (!cond.test(row))
> continue;
> {code}
> it results in unconditional building a joined row even if it will not be 
> emitted to downstream further. To avoid extra GC pressure we need to test the 
> join predicate before joining rows:
> {code:java}
> if (!cond.test(left, right))
> continue;
> Row row = handler.concat(left, right);
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)