[accumulo] branch main updated: Fate op defer time is not used (#2539)

[jm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jmark99 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new edbae8e  Fate op defer time is not used (#2539)
edbae8e is described below

commit edbae8e32cf1008ab898490dc00e776a56df09ab
Author: Mark Owens <jm...@apache.org>
AuthorDate: Thu Mar 3 07:46:07 2022 -0500

    Fate op defer time is not used (#2539)
    
    A comment was added to Fate.java to indicate how the deferTime variable
    is being used. It clarifies that the value is used in the manner of an
    exit code for the particular if/else loop after the comment.
    
    But it also makes clear that the value of deferTime is used as a wait
    value later on within the ZooStore class.
    
    No code was modified with this ticket.
    
    Co-authored-by: Christopher Tubbs <ct...@apache.org>
---
 core/src/main/java/org/apache/accumulo/fate/Fate.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/core/src/main/java/org/apache/accumulo/fate/Fate.java b/core/src/main/java/org/apache/accumulo/fate/Fate.java
index eeebb04..0ca8a0a 100644
--- a/core/src/main/java/org/apache/accumulo/fate/Fate.java
+++ b/core/src/main/java/org/apache/accumulo/fate/Fate.java
@@ -81,6 +81,9 @@ public class Fate<T> {
             try {
               deferTime = op.isReady(tid, environment);
 
+              // Here, deferTime is only used to determine success (zero) or failure (non-zero),
+              // proceeding on success and returning to the while loop on failure.
+              // The value of deferTime is only used as a wait time in ZooStore.unreserve
               if (deferTime == 0) {
                 prevOp = op;
                 if (status == TStatus.SUBMITTED) {