+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12755462/TEZ-2810.2.patch
against master revision 46decf3.
+1 @author. The patch does not contain any @author tags.
+1 tests included. The patch appears to include 1 new or modified test files.
+1 javac. The applied patch does not increase the total number of javac compiler warnings.
+1 javadoc. There were no new javadoc warning messages.
+1 findbugs. The patch does not introduce any new Findbugs (version 3.0.1) warnings.
+1 release audit. The applied patch does not increase the total number of release audit warnings.
+1 core tests. The patch passed unit tests in .
Test results: https://builds.apache.org/job/PreCommit-TEZ-Build/1118//testReport/
Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/1118//console
This message is automatically generated.
AnalyseDAG split into straggler analysis and allocation analysis.
Allocation analysis adds preemption support. Find all preempted attempts in the job. Identify those that were preempted while a critical path attempt was waiting for allocation. Mark that as a potential cause for the preemption.
Drawing critical path handles cases where allocation/launch have not occurred.
Added test for preemption logic.