search

hdinsight / presto-hdinsight

Presto on Azure HDInsight
3 stars 9 forks source link

fix infinity wait #9

Closed betarabbit closed 7 years ago

betarabbit commented 7 years ago

If there is something wrong happened during the installation, node may wait forever.

betarabbit commented 7 years ago

Hi @dharmeshkakadia , thanks for the review, I am an absolutely freshman in bash scripting, learned a lot from your comments. 😄 All changes are addressed, would you please help me review it again? Thanks!

betarabbit commented 7 years ago

Sorry, forgot to mention, I have tested it on Azure, it works as expected.

dharmeshkakadia commented 7 years ago

Thanks @BetaRabbit for your contribution !