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


##########
accord-core/src/main/java/accord/local/Status.java:
##########
@@ -76,7 +76,7 @@ public enum Status
      * To solve this problem we simply permit the executeAt we discover for B 
to be propagated to A* without
      * its dependencies. Though this does complicate the state machine a 
little.
      */
-    PreCommitted      (Accept,  Maybe, DefinitionUnknown, ExecuteAtKnown,   
DepsUnknown,  Unknown),
+    PreCommitted      (Accept,  Full,  DefinitionUnknown, ExecuteAtKnown,   
DepsUnknown,  Unknown),

Review Comment:
   Something changed. I decided to impose this to simplify remote callback 
handling, but it anyway reduces the number of edge cases in the state machine 
for minimal cost.



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