zabetak commented on code in PR #4625:
URL: https://github.com/apache/calcite/pull/4625#discussion_r2504474163
##########
core/src/main/java/org/apache/calcite/adapter/enumerable/NullPolicy.java:
##########
@@ -32,8 +32,6 @@ public enum NullPolicy {
STRICT,
/** Returns null if one of the arguments is null, and possibly other times.
*/
SEMI_STRICT,
- /** If any of the arguments are null, return null. */
- ANY,
Review Comment:
This is mostly used by code generators so I guess usages outside calcite
should be limited. Moreover, addressing the breaking change is rather
straightforward so I don't think end-users will complain much. Entries were
removed from this enum in the past and nobody complained about it.
If somebody feels strongly about deprecating first I am fine with it as well.
--
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]
