You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@calcite.apache.org by "Ruben Quesada Lopez (JIRA)" <ji...@apache.org> on 2019/06/07 16:23:00 UTC

[jira] [Updated] (CALCITE-3119) Deprecate Linq4j CorrelateJoinType (in favor of JoinType)

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

Ruben Quesada Lopez updated CALCITE-3119:
-----------------------------------------
    Description: 
CALCITE-2968 introduced in Linq4j a new (more general) enum JoinType. As a result, we have right now two enums for (almost) the same purpose: CorrelateJoinType (used for Correlation, thus defining only INNER, LEFT, SEMI, ANTI) and the new JoinType (can be used for any purpose, defining all types of join, mirroring core's JoinRelType).
The idea is to flag CorrelateJoinType as deprecated, and replace all its usages by JoinType.

  was:
CALCITE-2968 introduced in Linq4j a new (more general) JoinType. As a result, we have right now two enums for (almost) the same purpose: CorrelateJoinType (used for Correlation, thus defining only INNER, LEFT, SEMI, ANTI) and the new JoinType (can be used for any purpose, defining all types of join, mirroring core's JoinRelType).
The idea is to flag CorrelateJoinType as deprecated, and replace all its usages by JoinType.


> Deprecate Linq4j CorrelateJoinType (in favor of JoinType)
> ---------------------------------------------------------
>
>                 Key: CALCITE-3119
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3119
>             Project: Calcite
>          Issue Type: Task
>            Reporter: Ruben Quesada Lopez
>            Assignee: Ruben Quesada Lopez
>            Priority: Major
>             Fix For: 1.20.0
>
>
> CALCITE-2968 introduced in Linq4j a new (more general) enum JoinType. As a result, we have right now two enums for (almost) the same purpose: CorrelateJoinType (used for Correlation, thus defining only INNER, LEFT, SEMI, ANTI) and the new JoinType (can be used for any purpose, defining all types of join, mirroring core's JoinRelType).
> The idea is to flag CorrelateJoinType as deprecated, and replace all its usages by JoinType.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)