zzwqqq commented on code in PR #5385:
URL: https://github.com/apache/calcite/pull/5385#discussion_r4534553751
##########
core/src/main/java/org/apache/calcite/plan/RelOptUtil.java:
##########
@@ -287,6 +302,128 @@ public static List<CorrelationId>
getVariablesSetAndUsed(RelNode rel0,
return result;
}
+ /** Returns whether a relational expression, its inputs and sub-queries are
+ * deterministic according to their operator declarations.
+ *
+ * <p>Assumes parameters, correlation bindings, query context and table
contents
+ * stay fixed within the query. Function calls use
Review Comment:
Thanks. I've removed that wording. The definition now says that a
deterministic call returns the same result for the same operands within one
query execution.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]