Fix inverted existence check for hdfs/hadoop command

twitter-archive / commons

Twitter common libraries for python and the JVM (deprecated)

http://twitter.github.com/commons

Other

2.1k stars 565 forks source link

Fix inverted existence check for hdfs/hadoop command #442

Closed mleinart closed 7 years ago

mleinart commented 7 years ago

twitter.commons.fs.hdfs was broken by 18d4bbc8fb1c9fc5a2bcf9c3190aefcfcf36890b - When checking for the 'hdfs' or 'hadoop' commands, if the command does exist in the path the OSError is raised. Instead, the if statement needs to be inverted

BobbyD-FL commented 7 years ago

Sorry for the slip. @kwlzn please merge ^^ fix

mleinart commented 7 years ago

Submitted at 7e08b1f0b79d4b372f8c5a49defac79fd44b58b7