[
https://issues.apache.org/jira/browse/CALCITE-3008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16822035#comment-16822035
]
Rong Rong edited comment on CALCITE-3008 at 4/19/19 4:50 PM:
-------------------------------------------------------------
[UPDATE] did an experiment to do the public API change: please see:
https://github.com/walterddr/calcite/commit/9cce169ee93ed423d6cbb3aed6f6df018ccc8452
was (Author: walterddr):
Did a bit of look up, Seems like in this context:
https://github.com/apache/calcite/blob/branch-1.19/core/src/main/java/org/apache/calcite/sql2rel/SqlToRelConverter.java#L5150
it is already generic and it will require the SqlFunction to be encapsulated
into the list of AuxiliaryConverter objects.
> Making AuxiliaryConverter Pluggable
> -----------------------------------
>
> Key: CALCITE-3008
> URL: https://issues.apache.org/jira/browse/CALCITE-3008
> Project: Calcite
> Issue Type: Improvement
> Components: core
> Reporter: Rong Rong
> Priority: Major
> Labels: pull-request-available
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> Currently the {{AuxiliaryConverter}} in calcite core is set to be static with
> only one basic implementation. It would be nice to have it pluggable so that
> other systems can implement its own.
> Was wondering if this would be a good feature to add?
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)