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

Shuyi Chen commented on CALCITE-2171:
-------------------------------------

I see, do we leave it there just for demonstration purpose? But it's confusing 
though when I was reading the code. I would suggest clone the test to both use 
SPI and model.json file. Otherwise, if ModelHandler.addFunctions breaks, we 
wont be able to catch it. But maybe there are other tests testing the code path 
that I am unaware of. What do you think?

> ExampleFunctionTest is not reading model.json 
> ----------------------------------------------
>
>                 Key: CALCITE-2171
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2171
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Shuyi Chen
>            Assignee: Shuyi Chen
>            Priority: Major
>
> ExampleFunctionTest is not reading model.json at all. I think we can either 
> remove it, or modify the code to read model.json. Please let me know.



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

Reply via email to