dcapwell commented on code in PR #3995:
URL: https://github.com/apache/cassandra/pull/3995#discussion_r2006415170
##########
test/harry/main/org/apache/cassandra/harry/model/ASTSingleTableModel.java:
##########
@@ -893,28 +942,60 @@ private List<Clustering<ByteBuffer>> keys(Map<Symbol,
List<? extends Expression>
return
current.stream().map(BufferClustering::new).collect(Collectors.toList());
}
+ private static ByteBuffer eval(Symbol col, @Nullable ByteBuffer current,
Expression e)
+ {
+ if (!(e instanceof AssignmentOperator)) return eval(e);
+ // multi cell collections have the property that they do update even
if the current value is null
+ boolean isFancy = col.type().isCollection() &&
col.type().isMultiCell();
Review Comment:
i felt it was fancy!
--
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: pr-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
