[IGNITE-12256] Fix double invocation of javaMajorVersion in scripts

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.7.6
• Fix Version/s: 2.8
• Component/s: general
• Assignee: Ilya Kasnacheev
• Reporter: Ilya Kasnacheev
• Created: 03/Oct/19 11:19
• Updated: 07/Oct/19 11:12
• Resolved: 07/Oct/19 11:12

Description

Most of our shell script look as folows:

#
# Discover path to Java executable and check it's version.
#
checkJava

#
# Discover IGNITE_HOME environment variable.
#
setIgniteHome

#
# Final JVM_OPTS for Java 9+ compatibility
#
javaMajorVersion "${JAVA_HOME}/bin/java"

It makes no sense to me since we already call javaMajorVersion in checkJava. Let's try to get rid of it.

Issue Links

• duplicates: IGNITE-12208 If JAVA_HOME is not configured, the ignitevisorcmd.sh script cannot run
• relates to: IGNITE-9836 Invalid check of ea java versions
• links to: GitHub Pull Request #6932