[DRILL-5208] Finding path to java executable should be deterministic

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.10.0
• Fix Version/s: 1.10.0
• Component/s: Tools, Build & Test
• Labels: ready-to-commit
• Assignee: Paul Rogers
• Reporter: Krystal
• Reviewer: Arina Ielchiieva
• Created: 20/Jan/17 18:55
• Updated: 06/Mar/17 19:00
• Resolved: 03/Mar/17 22:15

Description

Command to find JAVA in drill-config.sh is not deterministic. drill-config.sh uses the following command to find JAVA:
JAVA=`find -L "$JAVA_HOME" -name $JAVA_BIN -type f | head -n 1`

On one of my node the following command returned 2 entries:
find -L $JAVA_HOME -name java -type f
/usr/local/java/jdk1.7.0_67/jre/bin/java
/usr/local/java/jdk1.7.0_67/bin/java

On another node, the same command returned entries in different order:
find -L $JAVA_HOME -name java -type f
/usr/local/java/jdk1.7.0_67/bin/java
/usr/local/java/jdk1.7.0_67/jre/bin/java

The complete command picks the first one returned which may not be the same on each node:
find -L $JAVA_HOME -name java -type f | head -n 1
/usr/local/java/jdk1.7.0_67/jre/bin/java

If JAVA_HOME is found, we should just append the "bin/java" to the path"
JAVA=$JAVA_HOME/bin/java

Issue Links

• links to: GitHub Pull Request #763