belliottsmith commented on code in PR #222:
URL: https://github.com/apache/cassandra-accord/pull/222#discussion_r2211534353


##########
accord-core/src/main/java/accord/impl/progresslog/DefaultProgressLog.java:
##########
@@ -333,7 +333,7 @@ public void clearBefore(SafeCommandStore safeStore, TxnId 
clearWaitingBefore, Tx
             {
                 // the command might be invalidated, which should be 
established on load, so simply load the command
                 TxnId txnId = state.txnId;
-                safeStore.commandStore().execute(txnId, safeStore0 -> {
+                
safeStore.commandStore().execute(PreLoadContext.contextFor(txnId, "Clear 
Progress"), safeStore0 -> {

Review Comment:
   not sure what you mean here, this one has a parameter - the `TxnId`. It is a 
shame we can't use the `TxnId` itself anymore, but probably better all round to 
have descriptions



-- 
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]

Reply via email to