[IGNITE-13136] Calcite integration. Improve join predicate testing.

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Component/s: sql
• Labels: calcite ignite-3
• Assignee: Evgeny Stanilovsky
• Reporter: Roman Kondakov
• Created: 09/Jun/20 12:58
• Updated: 16/Nov/21 15:56
• Resolved: 30/Jul/21 11:26

Description

Currently we have to merge joining rows in order to test a join predicate:

    Row row = handler.concat(left, rightMaterialized.get(rightIdx++));

    if (!cond.test(row))
        continue;

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:

    if (!cond.test(left, right))
        continue;

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

Issue Links

• is part of: IGNITE-12248 Apache Calcite based query execution engine
• links to: GitHub Pull Request #9264