DonnyZone commented on a change in pull request #1971:
URL: https://github.com/apache/calcite/pull/1971#discussion_r422758224
##########
File path:
core/src/main/java/org/apache/calcite/adapter/enumerable/NullPolicy.java
##########
@@ -25,6 +25,10 @@
* ANY whenever possible.</p>
*/
public enum NullPolicy {
+
+ /** Returns null if and only if all of the arguments are null;
+ * If all of the arguments are false return false otherwise true. */
Review comment:
> If all of the arguments are false return false otherwise true.
Do we support this? I think the policy is just for `null` value.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]