[ https://issues.apache.org/jira/browse/CALCITE-5349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17625287#comment-17625287 ]
TJ Banghart commented on CALCITE-5349: -------------------------------------- Hey Julian, yes that sounds like a better option. In that case we'd only need to change the lookup so all {{SqlLibrary}} operators are considered – we wouldn't need a new constructor or additional helper methods, etc... I can send something up shortly with this change. > RelJson should support non-standard operators > --------------------------------------------- > > Key: CALCITE-5349 > URL: https://issues.apache.org/jira/browse/CALCITE-5349 > Project: Calcite > Issue Type: Improvement > Components: core > Reporter: TJ Banghart > Assignee: TJ Banghart > Priority: Major > > I am attempting to deserialize {{RexNodes}} that contain non-standard > operators but {{RexJson#toOp}} only considers operators found in > {{{}SqlStdOperatorTable{}}}: > [https://github.com/apache/calcite/blob/c945b7f49b99538748c871557f6ac80957be2b6e/core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java#L780] > For example, attempting to deserialize a JSON RexCall that includes the > BigQuery only `DATE_FROM_UNIX_DATE` operator raises {{CalciteException: No > operator for 'DATE_FROM_UNIX_DATE' with kind: 'OTHER_FUNCTION', syntax: > 'FUNCTION' during JSON deserialization}} > Unless I am missing something, I don't believe the operator lookup is > configurable or overridable. -- This message was sent by Atlassian Jira (v8.20.10#820010)