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<>();