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

Jiatao Tao commented on CALCITE-4424:
-------------------------------------

Hi [~julianhyde], in fact, I'm also confused about the concept of dynamic 
functions. But I think they are orthogonal, so answer point2, I would not 
change that.

Func like CURRENT_DATE/CURRENT_TIMESTAMPE, I think it doesn't matter to 
determine them deterministic or non-deterministic, cuz current we do not 
observe that they will affect the optimized result(but rand will). So we just 
defined by our own standards, IMO we can define them as deterministic.

> Complete non-deterministic function in SqlFunction
> --------------------------------------------------
>
>                 Key: CALCITE-4424
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4424
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>            Reporter: Jiatao Tao
>            Assignee: Jiatao Tao
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Current only SqlSequenceValueOperator override isDeterministic, func like 
> RAND doesn't override isDeterministic, these func seems are not 
> deterministic(the list comes from "isDynamicFunction"):
> 1. SqlAbstractTimeFunction
> 2. SqlBaseContextVariable
> 3. SqlCurrentDateFunction
> 4. SqlRandFunction
> 5. SqlRandIntegerFunction
>  
>   



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to