(accumulo) branch main updated: Fix compaction prop log logic (#3987)

[ddanielr]

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

ddanielr 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 cb1a9ac937 Fix compaction prop log logic (#3987)
cb1a9ac937 is described below

commit cb1a9ac9373b0cae311e5a67021666e14b006430
Author: Daniel Roberts <ddani...@gmail.com>
AuthorDate: Fri Dec 8 14:58:19 2023 -0500

    Fix compaction prop log logic (#3987)
    
    Fixes an incorrect logging statement from appearing when processing
    `service.<service>.planner` compaction properties.
---
 .../accumulo/core/util/compaction/CompactionServicesConfig.java   | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git 
a/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
 
b/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
index 21a7719011..5e2f233a9a 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
@@ -133,13 +133,11 @@ public class CompactionServicesConfig {
           if (eprop == null || aconf.isPropertySet(eprop)) {
             rateLimits.put(tokens[0], 
ConfigurationTypeHelper.getFixedMemoryAsBytes(val));
           }
-        } else if (!(tokens.length == 2 && tokens[1].equals("planner"))) {
+        } else if (tokens.length == 2 && tokens[1].equals("planner")) {
+          return; // moves to next opt
+        } else {
           throw new IllegalArgumentException(
               "Malformed compaction service property " + prefix + prop);
-        } else {
-          log.warn(
-              "Ignoring compaction property {} as does not match the prefix 
used by the referenced planner definition",
-              prop);
         }
       });
     });