(accumulo) branch main updated: removes unneeded escaping in VolumeManagerIT (#6276)

[kturner]

This is an automated email from the ASF dual-hosted git repository.

kturner 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 fbe74b21b5 removes unneeded escaping in VolumeManagerIT (#6276)
fbe74b21b5 is described below

commit fbe74b21b50c96fe3043faa383ffa0b264ba6be9
Author: Keith Turner <ktur...@apache.org>
AuthorDate: Mon Mar 30 12:03:15 2026 -0700

    removes unneeded escaping in VolumeManagerIT (#6276)
    
    VolumeManagerIT was escaping key that would be stored in the
    accumulo.properties file.  After the changes in #6197 the keys were
    double escaped.  Removed the direct escaping in VolumeManagerIT as its
    no longer needed after the changes in #6197.
---
 .../java/org/apache/accumulo/test/VolumeManagerIT.java | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

diff --git a/test/src/main/java/org/apache/accumulo/test/VolumeManagerIT.java 
b/test/src/main/java/org/apache/accumulo/test/VolumeManagerIT.java
index 7e3e619936..161feb9958 100644
--- a/test/src/main/java/org/apache/accumulo/test/VolumeManagerIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/VolumeManagerIT.java
@@ -60,23 +60,17 @@ public class VolumeManagerIT extends ConfigurableMacBase {
       config.setProperty("general.custom.volume.preferred.default", vol1);
       config.setProperty("general.custom.volume.preferred.logger", vol2);
 
-      // Need to escape the colons in the custom property volume because it's 
part of the key. It's
-      // being written out to a file and read in using the Properties object.
-      String vol1FileOutput = vol1.replaceAll(":", "\\\\:");
-      String vol2FileOutput = vol2.replaceAll(":", "\\\\:");
-      config.setProperty(
-          Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + vol1FileOutput + 
".dfs.blocksize",
+      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + 
vol1 + ".dfs.blocksize",
           "10485760");
-      config.setProperty(
-          Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + vol2FileOutput + 
".dfs.blocksize",
+      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + 
vol2 + ".dfs.blocksize",
           "51200000");
-      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + 
vol1FileOutput
+      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + vol1
           + ".dfs.client.use.datanode.hostname", "true");
-      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + 
vol2FileOutput
+      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + vol2
           + ".dfs.client.use.datanode.hostname", "false");
-      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + 
vol1FileOutput
+      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + vol1
           + ".dfs.client.hedged.read.threadpool.size", "0");
-      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + 
vol2FileOutput
+      config.setProperty(Property.INSTANCE_VOLUME_CONFIG_PREFIX.getKey() + vol2
           + ".dfs.client.hedged.read.threadpool.size", "1");
     });
   }