arvindshmicrosoft commented on a change in pull request #294: Fix Muchos to 
work with Java 11
URL: https://github.com/apache/fluo-muchos/pull/294#discussion_r339249368
 
 

 ##########
 File path: lib/muchos/config.py
 ##########
 @@ -65,6 +65,11 @@ def verify_config(self, action):
                     if not self.has_service(service):
                         exit("ERROR - Missing '{0}' service from [nodes] 
section of muchos.props".format(service))
 
+            # validate if we are using Java 11 and fail if we are using 
Accumulo 1.x
+            # See https://github.com/apache/accumulo/issues/958 for details
+            if self.java_product_version() == 11 and 
self.version('accumulo').split('.')[0] == '1':
+                exit("ERROR - Java 11 is not supported with Accumulo version 
'{0}'".format(self.version('accumulo')))
 
 Review comment:
   Thanks Keith. We tested with 1.9.3 and Java 11, what failed was the build of 
the native libraries. Seemingly there is something in that Makefile which is 
tripping up, because it kept complaining about "no Java SDK at JAVA_HOME". That 
must be an Accumulo issue, I can open an issue on that project to track it 
separately.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to