You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by dl...@apache.org on 2024/11/20 13:31:58 UTC
(accumulo) branch 3.1 updated (d1527d4485 -> af2ca87aa4)
This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a change to branch 3.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git
from d1527d4485 Merge branch '2.1' into 3.1
add 943ca69fd6 Added Monitor property to optionally show extent information (#5081)
new af2ca87aa4 Merge branch '2.1' into 3.1
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../src/main/java/org/apache/accumulo/core/conf/Property.java | 10 +++++++++-
.../src/main/java/org/apache/accumulo/monitor/Monitor.java | 11 +++++++++++
.../accumulo/monitor/rest/tservers/TabletServerResource.java | 3 ++-
3 files changed, 22 insertions(+), 2 deletions(-)
(accumulo) 01/01: Merge branch '2.1' into 3.1
Posted by dl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a commit to branch 3.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git
commit af2ca87aa4c28db342b056564b9ce7ad864a5776
Merge: d1527d4485 943ca69fd6
Author: Dave Marion <dl...@apache.org>
AuthorDate: Wed Nov 20 13:31:42 2024 +0000
Merge branch '2.1' into 3.1
.../src/main/java/org/apache/accumulo/core/conf/Property.java | 10 +++++++++-
.../src/main/java/org/apache/accumulo/monitor/Monitor.java | 11 +++++++++++
.../accumulo/monitor/rest/tservers/TabletServerResource.java | 3 ++-
3 files changed, 22 insertions(+), 2 deletions(-)
diff --cc core/src/main/java/org/apache/accumulo/core/conf/Property.java
index 32d08169bc,0f1bf8ee33..a0d5399240
--- a/core/src/main/java/org/apache/accumulo/core/conf/Property.java
+++ b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
@@@ -878,6 -994,49 +878,14 @@@ public enum Property
+ " The resources that are used by default can be seen in"
+ " `accumulo/server/monitor/src/main/resources/templates/default.ftl`.",
"2.0.0"),
+ MONITOR_OBFUSCATE_EXTENTS("monitor.extents.obfuscate", "true", PropertyType.BOOLEAN,
+ "Obfuscates the table extent information displayed in the Monitor if true. Setting"
+ + " this to false will expose data in the row of the keys where tablets split. It"
+ + " is not recommended to set this to false if the Monitor is exposed to entities"
+ + " that should not see this information. The Monitor process reads this property"
+ + " from the accumulo.properties file only, not ZooKeeper, so it cannot be"
+ + " changed at runtime.",
+ "2.1.4"),
- @Deprecated(since = "2.1.0")
- TRACE_PREFIX("trace.", null, PropertyType.PREFIX,
- "Properties in this category affect the behavior of distributed tracing.", "1.3.5"),
- @Deprecated(since = "2.1.0")
- TRACE_SPAN_RECEIVERS("trace.span.receivers", "org.apache.accumulo.tracer.ZooTraceClient",
- PropertyType.CLASSNAMELIST, "A list of span receiver classes to send trace spans.", "1.7.0"),
- @Deprecated(since = "2.1.0")
- TRACE_SPAN_RECEIVER_PREFIX("trace.span.receiver.", null, PropertyType.PREFIX,
- "Prefix for span receiver configuration properties.", "1.7.0"),
- @Deprecated(since = "2.1.0")
- TRACE_ZK_PATH("trace.zookeeper.path", Constants.ZTRACERS, PropertyType.STRING,
- "The zookeeper node where tracers are registered.", "1.7.0"),
- @Deprecated(since = "2.1.0")
- TRACE_PORT("trace.port.client", "12234", PropertyType.PORT,
- "The listening port for the trace server.", "1.3.5"),
- @Deprecated(since = "2.1.0")
- TRACE_TABLE("trace.table", "trace", PropertyType.STRING,
- "The name of the table to store distributed traces.", "1.3.5"),
- @Deprecated(since = "2.1.0")
- TRACE_USER("trace.user", "root", PropertyType.STRING,
- "The name of the user to store distributed traces.", "1.3.5"),
- @Sensitive
- @Deprecated(since = "2.1.0")
- TRACE_PASSWORD("trace.password", "secret", PropertyType.STRING,
- "The password for the user used to store distributed traces.", "1.3.5"),
- @Sensitive
- @Deprecated(since = "2.1.0")
- TRACE_TOKEN_PROPERTY_PREFIX("trace.token.property.", null, PropertyType.PREFIX,
- "The prefix used to create a token for storing distributed traces. For"
- + " each property required by trace.token.type, place this prefix in front of it.",
- "1.5.0"),
- @Deprecated(since = "2.1.0")
- TRACE_TOKEN_TYPE("trace.token.type", PasswordToken.class.getName(), PropertyType.CLASSNAME,
- "An AuthenticationToken type supported by the authorizer.", "1.5.0"),
-
// per table properties
TABLE_PREFIX("table.", null, PropertyType.PREFIX,
"Properties in this category affect tablet server treatment of tablets,"
diff --cc server/monitor/src/main/java/org/apache/accumulo/monitor/Monitor.java
index f25aa8f545,315b107e88..0c1b780bbc
--- a/server/monitor/src/main/java/org/apache/accumulo/monitor/Monitor.java
+++ b/server/monitor/src/main/java/org/apache/accumulo/monitor/Monitor.java
@@@ -115,16 -115,23 +115,23 @@@ public class Monitor extends AbstractSe
private static final int REFRESH_TIME = 5;
private final long START_TIME;
+ private final boolean obfuscateExtents;
public static void main(String[] args) throws Exception {
- try (Monitor monitor = new Monitor(new ServerOpts(), args)) {
+ try (Monitor monitor = new Monitor(new ConfigOpts(), args)) {
monitor.runServer();
}
}
- Monitor(ServerOpts opts, String[] args) {
+ Monitor(ConfigOpts opts, String[] args) {
super("monitor", opts, args);
START_TIME = System.currentTimeMillis();
+ // Using site configuration on purpose. We want to get the value from
+ // accumulo.properties file local to the Monitor. We don't want to
+ // enable someone to change this property dynamically and expose
+ // information.
+ obfuscateExtents =
+ getContext().getSiteConfiguration().getBoolean(Property.MONITOR_OBFUSCATE_EXTENTS);
}
private final AtomicLong lastRecalc = new AtomicLong(0L);