[exec] -1 overall.
[exec] +1 @author. The patch does not contain any @author tags.
[exec] -1 tests included. The patch doesn't appear to include any new or modified tests.
[exec] Please justify why no new tests are needed for this patch.
[exec] Also please list what manual steps were performed to verify this patch.
[exec] +1 javadoc. The javadoc tool did not generate any warning messages.
[exec] -1 javac. The applied patch generated 64 javac compiler warnings (more than the trunk's current 124 warnings).
[exec] +1 findbugs. The patch does not introduce any new Findbugs warnings.
[exec] +1 release audit. The applied patch does not increase the total number of release audit warnings.
There are no tests for the current EC2 scripts, so I tested this manually by running through starting a cluster, running a job, and shutting the cluster down. I tested a failure by setting an invalid key name.
The -1 for javac is odd, since the number went down according to the message, and furthermore this patch doesn't change any Java code.