(accumulo) branch elasticity updated: Modified TabletGoalState log msg to remove todo (#4406)

[dlmarion]

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

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


The following commit(s) were added to refs/heads/elasticity by this push:
     new 6176c3d07c Modified TabletGoalState log msg to remove todo (#4406)
6176c3d07c is described below

commit 6176c3d07cd57d9e208a6bb8e2580a2e62ae8545
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Fri Mar 22 08:38:01 2024 -0400

    Modified TabletGoalState log msg to remove todo (#4406)
---
 .../accumulo/server/manager/state/TabletGoalState.java      | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/manager/state/TabletGoalState.java
 
b/server/base/src/main/java/org/apache/accumulo/server/manager/state/TabletGoalState.java
index 0b9b83f159..ca9b2ada8c 100644
--- 
a/server/base/src/main/java/org/apache/accumulo/server/manager/state/TabletGoalState.java
+++ 
b/server/base/src/main/java/org/apache/accumulo/server/manager/state/TabletGoalState.java
@@ -146,15 +146,10 @@ public enum TabletGoalState {
             return UNASSIGNED;
           }
         } else {
-          // ELASTICITY_TODO this log level was set to error so that this case 
can be examined for
-          // bugs. A tablet server should always have a resource group. If 
there are unavoidable
-          // race conditions for getting tablet servers and their RGs, that 
that should be handled
-          // in the TabletManagementParameters data acquisition phase so that 
not all code has to
-          // deal with it. Eventually this log level should possibly be 
adjusted or converted to an
-          // exception.
-          log.error(
-              "Could not find resource group for tserver {}, so did not 
consult balancer.  Need to determine the cause of this.",
-              tm.getLocation().getServerInstance());
+          log.warn("Could not find resource group for tserver {}, did not 
consult balancer to"
+              + " check if tablet {} needs to be re-assigned. This tablet will 
be rechecked"
+              + " soon. If this condition is not transient, then it could 
indicate a bug so"
+              + " please report it.", tm.getLocation().getServerInstance(), 
tm.getExtent());
         }
       }