adelapena commented on code in PR #2386:
URL: https://github.com/apache/cassandra/pull/2386#discussion_r1222889014
##########
src/java/org/apache/cassandra/cql3/functions/ScalarFunction.java:
##########
@@ -66,19 +65,19 @@ public default boolean isMonotonic()
* </pre>
* and such that for any value of {@code b} and {@code d}, {@code bar(b,
d) == foo(3, b, 'bar', d)}.
*
- * @param protocolVersion protocol version used for parameters
- * @param partialParameters a list of input parameters for the function
where some parameters can be {@link #UNRESOLVED}.
- * The input <b>must</b> be of size {@code
this.argsType().size()}. For convenience, it is
- * allowed both to pass a list with all
parameters being {@link #UNRESOLVED} (the function is
- * then returned directly) and with none of them
unresolved (in which case, if the function is pure,
- * it is computed and a dummy no-arg function
returning the result is returned).
+ * @param protocolVersion protocol version used for arguments
+ * @param partialArguments a list of input arguments for the function
where some arguments can be {@link #UNRESOLVED}.
+ * The input <b>must</b> be of size {@code
this.argsType().size()}. For convenience, it is
+ * allowed both to pass a list with all parameters
being {@link #UNRESOLVED} (the function is
+ * then returned directly) and with none of them
unresolved (in which case, if the function is pure,
+ * it is computed and a dummy no-arg function
returning the result is returned).
* @return a function corresponding to the partial application of this
function to the parameters of
- * {@code partialParameters} that are not {@link #UNRESOLVED}.
+ * {@code partialArguments} that are not {@link #UNRESOLVED}.
*/
- public default ScalarFunction partialApplication(ProtocolVersion
protocolVersion, List<ByteBuffer> partialParameters)
+ default ScalarFunction partialApplication(ProtocolVersion protocolVersion,
List<ByteBuffer> partialArguments)
{
int unresolvedCount = 0;
- for (ByteBuffer parameter : partialParameters)
+ for (ByteBuffer parameter : partialArguments)
Review Comment:
Right, changed.
##########
src/java/org/apache/cassandra/cql3/selection/AbstractFunctionSelector.java:
##########
@@ -74,19 +78,27 @@ protected Selector deserialize(DataInputPlus in, int
version, TableMetadata meta
.collect(joining(", "))));
boolean isPartial = in.readBoolean();
+ // if the function is partial we need to retrieve the resolved
parameters.
Review Comment:
Indeed we should, done.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
