This is an automated email from the ASF dual-hosted git repository.
krathbun 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 dbd2d50a7e Minor follow on to FATE config changes #5861 (#5884)
dbd2d50a7e is described below
commit dbd2d50a7e3eadee7dce5bbed6e5077532017ffb
Author: Kevin Rathbun <[email protected]>
AuthorDate: Mon Sep 15 10:34:17 2025 -0400
Minor follow on to FATE config changes #5861 (#5884)
Updates descriptions for FATE config in PropertyType. These appear on
the server-properties page on the Accumulo website.
---
.../apache/accumulo/core/conf/PropertyType.java | 26 +++++++++++-----------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java
b/core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java
index 1af98f7b72..27a61ef82a 100644
--- a/core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java
+++ b/core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java
@@ -161,19 +161,8 @@ public enum PropertyType {
"One of the currently supported filename extensions for storing table
data files. "
+ "Currently, only " + RFile.EXTENSION + " is supported."),
VOLUMES("volumes", new ValidVolumes(), "See instance.volumes documentation"),
- FATE_USER_CONFIG(ValidUserFateConfig.NAME, new ValidUserFateConfig(),
- "An arbitrary string that: 1. Represents a valid, parsable generic json
object. "
- + "2. the keys of the json are strings which contain a
comma-separated list of fate operations. "
- + "3. the values of the json are integers which represent the number
of threads assigned to the fate operations. "
- + "4. all possible user fate operations are present in the json. "
- + "5. no fate operations are repeated."),
-
- FATE_META_CONFIG(ValidMetaFateConfig.NAME, new ValidMetaFateConfig(),
- "An arbitrary string that: 1. Represents a valid, parsable generic json
object. "
- + "2. the keys of the json are strings which contain a
comma-separated list of fate operations. "
- + "3. the values of the json are integers which represent the number
of threads assigned to the fate operations. "
- + "4. all possible meta fate operations are present in the json. "
- + "5. no fate operations are repeated."),
+ FATE_USER_CONFIG(ValidUserFateConfig.NAME, new ValidUserFateConfig(),
ValidUserFateConfig.DESC),
+ FATE_META_CONFIG(ValidMetaFateConfig.NAME, new ValidMetaFateConfig(),
ValidMetaFateConfig.DESC),
FATE_THREADPOOL_SIZE("(deprecated) Manager FATE thread pool size", new
FateThreadPoolSize(),
"No format check. Allows any value to be set but will warn the user that
the"
+ " property is no longer used."),
@@ -474,6 +463,15 @@ public enum PropertyType {
}
private static class ValidFateConfig implements Predicate<String> {
+ private static final String DESC = """
+ A string that:
+ 1. Represents a valid, parsable JSON object.
+ 2. Each key is a string which represents a name for the value/pool.
+ 3. Each value is a JSON with a single key/value: key = comma-separated
string list of FATE
+ operations, value = integer for the number of threads assigned to run
those FATE operations.
+ 4. All possible %s FATE operations are present in the JSON.
+ 5. No FATE operations are repeated.
+ """;
private static final Logger log =
LoggerFactory.getLogger(ValidFateConfig.class);
private final Set<Fate.FateOperation> allFateOps;
private final String name;
@@ -549,6 +547,7 @@ public enum PropertyType {
}
private static class ValidUserFateConfig extends ValidFateConfig {
+ private static final String DESC = String.format(ValidFateConfig.DESC,
"user");
private static final String NAME = "fate user config";
private ValidUserFateConfig() {
@@ -557,6 +556,7 @@ public enum PropertyType {
}
private static class ValidMetaFateConfig extends ValidFateConfig {
+ private static final String DESC = String.format(ValidFateConfig.DESC,
"meta");
private static final String NAME = "fate meta config";
private ValidMetaFateConfig() {