dcapwell commented on code in PR #4044:
URL: https://github.com/apache/cassandra/pull/4044#discussion_r2032081520
##########
test/unit/org/apache/cassandra/cql3/ast/Mutation.java:
##########
@@ -161,22 +166,38 @@ public Stream<? extends Element> stream()
{
return Stream.of(value);
}
+
+ public long get()
+ {
+ if (value.value() instanceof Long)
+ return (long) value.value();
+ return LongType.instance.compose(value.valueEncoded());
+ }
}
public static class Using implements Element
{
public final Optional<TTL> ttl;
public final Optional<Timestamp> timestamp;
- public Using(Optional<TTL> ttl, Optional<Timestamp> timestamp)
+ private Using(Optional<TTL> ttl, Optional<Timestamp> timestamp)
{
this.ttl = ttl;
this.timestamp = timestamp;
+ if (ttl.isEmpty() && timestamp.isEmpty())
Review Comment:
this isn't CQL parsing code, its an isolated `ast` that is used to generate
CQL. This can't break production
--
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
