belliottsmith commented on code in PR #113:
URL: https://github.com/apache/cassandra-accord/pull/113#discussion_r1746059337
##########
accord-core/src/main/java/accord/local/cfk/Pruning.java:
##########
@@ -195,22 +204,27 @@ static CommandsForKey maybePrune(CommandsForKey cfk, int
pruneInterval, long min
/**
* We can prune anything transitively applied where some later stable
command replicates each of its missing array entries.
* These later commands can durably stand in for any recovery or
dependency calculations.
+ *
+ * TODO (desired): we could limit this restriction to epochs where
ownership changes; introduce some global summary info to facilitate this
*/
static CommandsForKey pruneBefore(CommandsForKey cfk, TxnInfo
newPrunedBefore, int pos)
Review Comment:
Yes, the class level doc of `CommandsForKey` explains this all in some detail
--
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]
