You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2019/06/14 04:46:05 UTC

[accumulo] branch 2.0 updated: Fix tests broken due to #1202

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

ctubbsii pushed a commit to branch 2.0
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.0 by this push:
     new 190842a  Fix tests broken due to #1202
190842a is described below

commit 190842aaa92c1fdeb98e11a0756644457ea73fa3
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Fri Jun 14 00:44:54 2019 -0400

    Fix tests broken due to #1202
---
 core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

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 6ce1120..64b19c8 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
@@ -102,8 +102,10 @@ public enum PropertyType {
           + "Substitutions of the ACCUMULO_HOME environment variable can be done in the system "
           + "config file using '${env:ACCUMULO_HOME}' or similar."),
 
+  // VFS_CLASSLOADER_CACHE_DIR's default value is a special case, for documentation purposes
   ABSOLUTEPATH("absolute path",
-      x -> x == null || x.trim().isEmpty() || new Path(x.trim()).isAbsolute(),
+      x -> x == null || x.trim().isEmpty() || new Path(x.trim()).isAbsolute()
+          || x.equals(Property.VFS_CLASSLOADER_CACHE_DIR.getDefaultValue()),
       "An absolute filesystem path. The filesystem depends on the property."
           + " This is the same as path, but enforces that its root is explicitly" + " specified."),