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_r339301210
##########
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:
I didn't even remember that, thanks for the link. One of my goals is to
make 1.9.4 work with Java 11 in order to provide a more gradual upgrade path.
People can upgrade to Java 11 while still using Accumulo 1.9, not having to
upgrade to Java 11, Hadoop 3, and Accumulo 2 all at once. I did a round of
testing with Java 11 in the past and just fixed whatever issues I found.
Hopefully I can do that again before 1.9.4 is released.
----------------------------------------------------------------
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