gianm commented on a change in pull request #9893:
URL: https://github.com/apache/druid/pull/9893#discussion_r428780412



##########
File path: 
sql/src/main/java/org/apache/druid/sql/calcite/expression/OperatorConversions.java
##########
@@ -291,15 +294,15 @@ public OperatorBuilder operandTypes(final 
SqlTypeFamily... operandTypes)
       return this;
     }
 
-    public OperatorBuilder operandTypeInference(final SqlOperandTypeInference 
operandTypeInference)
+    public OperatorBuilder requiredOperands(final int requiredOperands)
     {
-      this.operandTypeInference = operandTypeInference;
+      this.requiredOperands = requiredOperands;
       return this;
     }
 
-    public OperatorBuilder requiredOperands(final int requiredOperands)
+    public OperatorBuilder literalOperands(final int... literalOperands)

Review comment:
       I added javadocs to all these methods.




----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to