You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/02/04 17:43:35 UTC

[GitHub] [accumulo] milleruntime commented on a change in pull request #2462: Added FaTE TStatus.SUBMITTED

milleruntime commented on a change in pull request #2462:
URL: https://github.com/apache/accumulo/pull/2462#discussion_r799680404



##########
File path: core/src/main/java/org/apache/accumulo/fate/Fate.java
##########
@@ -284,7 +287,7 @@ public void seedTransaction(long tid, Repo<T> repo, boolean autoCleanUp) {
 
         store.setProperty(tid, DEBUG_PROP, repo.getDescription());
 
-        store.setStatus(tid, TStatus.IN_PROGRESS);
+        store.setStatus(tid, TStatus.SUBMITTED);

Review comment:
       I think there are some OPs that do work in the `isReady()` function so I wonder if changing when it gets set to IN_PROGRESS will affect them. I will see if I can find one.




-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org