You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Konstantin Orlov (Jira)" <ji...@apache.org> on 2022/11/15 11:34:00 UTC

[jira] [Commented] (IGNITE-17704) Get rid of usage of RelDataType in execution nodes

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

Konstantin Orlov commented on IGNITE-17704:
-------------------------------------------

[~xtern], LGTM!

> Get rid of usage of RelDataType in execution nodes
> --------------------------------------------------
>
>                 Key: IGNITE-17704
>                 URL: https://issues.apache.org/jira/browse/IGNITE-17704
>             Project: Ignite
>          Issue Type: Improvement
>          Components: sql
>            Reporter: Yury Gerzhedovich
>            Assignee: Pavel Pereslegin
>            Priority: Major
>              Labels: ignite-3, tech-debt
>             Fix For: 3.0.0-beta2
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Execution-related classes have no third-party dependencies except for apache calcite. The only thing connecting them is the RelDataType class, describing a type of the row.
> Seems the only actual usage of row type is inside {{CorrelatedNestedLoopJoinNode}}.
> So let's get rid of unnecessary coupling by reworking {{CorrelatedNestedLoopJoinNode}} class.



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