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

dlmarion 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 08dc1fd6a6 Fail if config cmd used with rg but not -s or -d (#5862)
08dc1fd6a6 is described below

commit 08dc1fd6a67050ca1d5f57c2557b74116a0cd519
Author: Dave Marion <[email protected]>
AuthorDate: Tue Sep 16 06:19:09 2025 -0400

    Fail if config cmd used with rg but not -s or -d (#5862)
    
    The config command will print all matching resource
    group properties by default. There is no need for
    the resource group to be specified unless setting
    or deleting a property.
    
    Related to #5859
---
 .../main/java/org/apache/accumulo/shell/commands/ConfigCommand.java | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java 
b/shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java
index 19fc07e2ad..a21d5952b2 100644
--- a/shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java
+++ b/shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java
@@ -213,6 +213,12 @@ public class ConfigCommand extends Command {
         logSysPropChanged(theProp, "set");
       }
     } else {
+
+      if (rgid != null) {
+        throw new IllegalArgumentException(
+            "resource group argument must be accompanied by -s or -d 
argument.");
+      }
+
       boolean warned = false;
       // display properties
       final TreeMap<String,String> systemConfig = new TreeMap<>();

Reply via email to