smiklosovic commented on code in PR #1954:
URL: https://github.com/apache/cassandra/pull/1954#discussion_r1052288718
##########
src/java/org/apache/cassandra/db/Mutation.java:
##########
@@ -64,6 +66,13 @@ public class Mutation implements IMutation,
Supplier<Mutation>
private final boolean cdcEnabled;
+ // Contains serialized representations of this mutation.
+ // Note: there is no functionality to clear/remove serialized instances,
because a mutation must never
+ // be modified (e.g. calling add(PartitionUpdate)) when it's being
serialized.
+ private static final int CACHED_SERIALIZATIONS =
MessagingService.Version.values().length;
+ private static final int CACHEABLE_MUTATION_SIZE_LIMIT =
Integer.getInteger(Config.PROPERTY_PREFIX +
"cacheable_mutation_size_limit_bytes", 2 * 1024 * 1024) - 24;
Review Comment:
This should be moved to `CassandraRelevantProperties`
--
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]
