This is an automated email from the ASF dual-hosted git repository.
ddanielr 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 9261f6aff6 Updates command description statements (#6411)
9261f6aff6 is described below
commit 9261f6aff65f8f11b3ff75bb158a20701605ecff
Author: Daniel Roberts <[email protected]>
AuthorDate: Mon Jun 8 11:34:15 2026 -0400
Updates command description statements (#6411)
* Updates command description statements
Updates the ./accumulo zookeeper command description to be less vauge
about the "Zookeeper instance".
Updates docs to clarify split point selection
---------
Co-authored-by: Christopher Tubbs <[email protected]>
---
.../java/org/apache/accumulo/core/client/admin/TableOperations.java | 4 +++-
.../main/java/org/apache/accumulo/server/util/ZooKeeperMain.java | 2 +-
.../java/org/apache/accumulo/shell/commands/GetSplitsCommand.java | 6 +++---
3 files changed, 7 insertions(+), 5 deletions(-)
diff --git
a/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
b/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
index b59e9855bd..3b4fed9637 100644
---
a/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
+++
b/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
@@ -269,7 +269,9 @@ public interface TableOperations {
/**
* @param tableName the name of the table
- * @param maxSplits specifies the maximum number of splits to return
+ * @param maxSplits specifies the maximum number of selected splits to
return. The selection of
+ * split points is defined by the implementation and may not always
be in immediate
+ * sequential order.
* @throws AccumuloException if a general error occurs
* @throws AccumuloSecurityException if the user does not have permission
* @return the split points (end-row names) for the table's current split
profile, grouped into
diff --git
a/server/base/src/main/java/org/apache/accumulo/server/util/ZooKeeperMain.java
b/server/base/src/main/java/org/apache/accumulo/server/util/ZooKeeperMain.java
index b98ef90b34..b181a452b2 100644
---
a/server/base/src/main/java/org/apache/accumulo/server/util/ZooKeeperMain.java
+++
b/server/base/src/main/java/org/apache/accumulo/server/util/ZooKeeperMain.java
@@ -56,7 +56,7 @@ public class ZooKeeperMain implements KeywordExecutable {
@Override
public String description() {
- return "Starts Apache Zookeeper instance";
+ return "Starts an Apache ZooKeeper client rooted at the current instance";
}
@Override
diff --git
a/shell/src/main/java/org/apache/accumulo/shell/commands/GetSplitsCommand.java
b/shell/src/main/java/org/apache/accumulo/shell/commands/GetSplitsCommand.java
index 1cef6d3151..f71e3325b6 100644
---
a/shell/src/main/java/org/apache/accumulo/shell/commands/GetSplitsCommand.java
+++
b/shell/src/main/java/org/apache/accumulo/shell/commands/GetSplitsCommand.java
@@ -90,7 +90,7 @@ public class GetSplitsCommand extends Command {
if (TabletColumnFamily.PREV_ROW_COLUMN.hasColumns(next.getKey())) {
KeyExtent extent = KeyExtent.fromMetaPrevRow(next);
final String obscured = extent.obscured();
- if (matches.size() == 0 || matches.contains(obscured)) {
+ if (matches.isEmpty() || matches.contains(obscured)) {
final String pr = encode(encode, extent.prevEndRow());
final String er = encode(encode, extent.endRow());
final String line =
@@ -139,8 +139,8 @@ public class GetSplitsCommand extends Command {
outputFileOpt = new Option("o", "output", true, "local file to write the
splits to");
outputFileOpt.setArgName("file");
- maxSplitsOpt =
- new Option("m", "max", true, "maximum number of splits to return
(evenly spaced)");
+ maxSplitsOpt = new Option("m", "max", true, "maximum number of splits to
return that are"
+ + " selected from the existing splits; see
TableOperations.listSplits");
maxSplitsOpt.setArgName("num");
base64Opt = new Option("b64", "base64encoded", false, "encode the split
points");
