keith-turner 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_r339247674
##########
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:
My memory is a bit fuzzy on this. I thought some work was done to make 1.9
work with Java 11. Maybe that will be for 1.9.4. This is fine for now, may
want to revisit this in the future.
----------------------------------------------------------------
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
