(accumulo) branch 2.1 updated: Adjusts compactor error logging because of race (#4151)

[kturner]

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

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


The following commit(s) were added to refs/heads/2.1 by this push:
     new 896e6277e1 Adjusts compactor error logging because of race (#4151)
896e6277e1 is described below

commit 896e6277e1e6dfd13166fd1f5ebf9eba5994027f
Author: Keith Turner <ktur...@apache.org>
AuthorDate: Thu Feb 8 13:55:56 2024 -0500

    Adjusts compactor error logging because of race (#4151)
    
    Noticed error logging from the compactor process and found it was caused
    by a race condition.  The logging indicated a compaction was not
    running when it was expected to, but the compaction started shortly
    after.  The background thread had not yet populated a set that the
    foreground thread was examining.  Adjusted the error logging to debug
    since it can happen spuriously.
---
 .../src/main/java/org/apache/accumulo/compactor/Compactor.java          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java 
b/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
index 823f9fd58d..893069e920 100644
--- 
a/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
+++ 
b/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
@@ -701,7 +701,7 @@ public class Compactor extends AbstractServer implements 
MetricsProducer, Compac
                 }
               }
             } else {
-              LOG.error("Waiting on compaction thread to finish, but no 
RUNNING compaction");
+              LOG.debug("Waiting on compaction thread to finish, but no 
RUNNING compaction");
             }
           }
           compactionThread.join();