[STORM-3011] Use default bin path in flight.bash if $JAVA_HOME is undefined

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.2.1
• Fix Version/s: 2.0.0, 1.1.3, 1.0.7, 1.2.2
• Labels: pull-request-available
• Assignee: Julien Nioche
• Reporter: Julien Nioche
• Created: 27/Mar/18 13:40
• Updated: 06/Apr/18 15:15
• Resolved: 30/Mar/18 14:25

Description

I had to modify the flight.bash script as $JAVA_HOME is undefined, this yields
 
/usr/share/apache-storm-1.2.1/bin/flight.bash: line 62: /bin/jstack: No such file or directory
 
however, the  files can be found in the default path /usr/bin
 
I will submit a PR so that the script checks that $JDKPATH is not empty and exist before using it for BINPATH. I can't see any downsides in doing so.

Issue Links

• links to: GitHub Pull Request #2607
• links to: GitHub Pull Request #2609