You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Julian Hyde (Jira)" <ji...@apache.org> on 2023/02/10 17:07:00 UTC

[jira] [Resolved] (CALCITE-5514) Add a public toRex() instance method for RelJson

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

Julian Hyde resolved CALCITE-5514.
----------------------------------
    Resolution: Fixed

Fixed in [a9587afd|https://github.com/apache/calcite/commit/a9587afd194ddbce2a9855ef054406e35f599518]; thanks for the PR, [~tjbanghart]!

> Add a public toRex() instance method for RelJson
> ------------------------------------------------
>
>                 Key: CALCITE-5514
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5514
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.32.0
>            Reporter: TJ Banghart
>            Assignee: TJ Banghart
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.34.0
>
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> CALCITE-5349 added several improvements for creating {{org.apache.calcite.rel.externalize.RelJson}} such as the {{withOperatorTable, withLibraryOperatorTable}} and, {{withJsonBuilder}} methods. 
> However, the only method to deserialize JSON with the {{RelJson}} class is a the {{static}} {{readExpression()}} which creates a generic {{RelJson}}. 
> It would be advantageous to have a {{RelJson}} instance method capable of deserializing JSON into a {{RexNode}} after it has been customized.



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