Skip to content

[CALCITE-4424] Complete non-deterministic function (Jiatao Tao)#2284

Open
Aaaaaaron wants to merge 1 commit intoapache:mainfrom
Aaaaaaron:CALCITE-4424
Open

[CALCITE-4424] Complete non-deterministic function (Jiatao Tao)#2284
Aaaaaaron wants to merge 1 commit intoapache:mainfrom
Aaaaaaron:CALCITE-4424

Conversation

@Aaaaaaron
Copy link
Copy Markdown
Member

@Aaaaaaron Aaaaaaron commented Dec 2, 2020

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

@Aaaaaaron Aaaaaaron force-pushed the CALCITE-4424 branch 3 times, most recently from bf9091d to 1782954 Compare December 2, 2020 08:27
@Aaaaaaron
Copy link
Copy Markdown
Member Author

Aaaaaaron commented Dec 2, 2020

image
Seems an ustable UT, re-run passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

discussion-in-jira There's open discussion in JIRA to be resolved before proceeding with the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants