[GitHub] [accumulo] dlmarion commented on a change in pull request #2414: Fix issues in MiniAccumuloCluster.verifyUp that is causing tests to fail

[GitBox <gi...@apache.org>]

dlmarion commented on a change in pull request #2414:
URL: https://github.com/apache/accumulo/pull/2414#discussion_r788967393



##########
File path: minicluster/src/main/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImpl.java
##########
@@ -327,6 +334,13 @@ public MiniAccumuloClusterImpl(MiniAccumuloConfigImpl config) throws IOException
 
     this.config = config.initialize();
 
+    if (Boolean.TRUE.equals(Boolean
+        .valueOf(this.config.getSiteConfig().get(Property.TSERV_NATIVEMAP_ENABLED.name())))) {
+      if (!NativeMap.isLoaded())
+        throw new RuntimeException(
+            "MAC configured to use native maps, but unable to load the library.");
+    }
+

Review comment:
       `ConfigurableMacBase.createMiniAccumulo` sets `TSERV_NATIVEMAP_ENABLED` to true and then starts MAC. This checks that NativeMap will be able to load the library in this test thread before starting up the processes.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org