This is an automated email from the ASF dual-hosted git repository.
kturner pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/elasticity by this push:
new 8798fccd5c updates IdleProcessMetricsIT to verify config instead of
set it. (#4331)
8798fccd5c is described below
commit 8798fccd5c32874cfd2d8f255781820280534e0f
Author: Keith Turner <[email protected]>
AuthorDate: Fri Mar 1 13:42:50 2024 -0500
updates IdleProcessMetricsIT to verify config instead of set it. (#4331)
IdleProcessMetricsIT was setting compactor config to the same as the
default. Changed the test to verify the config instead of setting it.
---
.../apache/accumulo/test/functional/IdleProcessMetricsIT.java | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git
a/test/src/main/java/org/apache/accumulo/test/functional/IdleProcessMetricsIT.java
b/test/src/main/java/org/apache/accumulo/test/functional/IdleProcessMetricsIT.java
index c2d2f649ae..36d16b8d2d 100644
---
a/test/src/main/java/org/apache/accumulo/test/functional/IdleProcessMetricsIT.java
+++
b/test/src/main/java/org/apache/accumulo/test/functional/IdleProcessMetricsIT.java
@@ -18,11 +18,14 @@
*/
package org.apache.accumulo.test.functional;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
import java.time.Duration;
import java.util.List;
import java.util.Map;
import java.util.concurrent.atomic.AtomicBoolean;
+import org.apache.accumulo.core.Constants;
import org.apache.accumulo.core.conf.Property;
import org.apache.accumulo.core.metrics.MetricsProducer;
import org.apache.accumulo.harness.MiniClusterConfigurationCallback;
@@ -42,10 +45,10 @@ public class IdleProcessMetricsIT extends
SharedMiniClusterBase {
@Override
public void configureMiniCluster(MiniAccumuloConfigImpl cfg, Configuration
coreSite) {
- // Configure all compaction planners to use the default resource group so
- // that only 1 compactor is started by MiniAccumuloCluster
- cfg.setProperty(Property.COMPACTION_SERVICE_DEFAULT_GROUPS.getKey(),
- "[{'group':'default'}]".replaceAll("'", "\""));
+ // Verify expectations about the default config. Want to ensure there no
other resource groups
+ // configured.
+ assertEquals(Map.of(Constants.DEFAULT_COMPACTION_SERVICE_NAME, 1),
+ cfg.getClusterServerConfiguration().getCompactorConfiguration());
// Disable the default scan servers and compactors, just start 1
// tablet server in the default group to host the root and metadata
