okay ! heres a patch to upgrade smoke-tests so that they work again. There were a bug in the Failure Executor that needed to be patched, and also, i needed to add source dependencies on itest.
This is much better ! *it means that when we run gradle smoke-test recipes, we actually use iTest source - so it provides further automated validation of the bigtop codebase. It was also necessary since the introduction of the failureVars, since those arent published to Itest.
- There was a minor bug in FailureExecutor which caused booleans to be parsed as "true" on accident .
- This fixes that, and also updates the smoke-tests.sh script in vagrant recipe to run clean before testing.
- Finally, it also adds iTest to the source path, so that smoke tests (for mapreduce) no longer require an ITest jar.
Later we can upgrade all smokes (pig,mahout,hive...) to use the itest sources on maste rather than jarfiler, but for now, this is the minimal patch (mapreduce) to fix the problem.
looks okay ?