You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by hi...@apache.org on 2016/06/08 02:50:19 UTC

tez git commit: TEZ-2846. Flaky test: TestCommit.testVertexCommit_OnDAGSuccess. (hitesh)

Repository: tez
Updated Branches:
  refs/heads/master a2d120b22 -> 05d7725d2


TEZ-2846. Flaky test: TestCommit.testVertexCommit_OnDAGSuccess. (hitesh)


Project: http://git-wip-us.apache.org/repos/asf/tez/repo
Commit: http://git-wip-us.apache.org/repos/asf/tez/commit/05d7725d
Tree: http://git-wip-us.apache.org/repos/asf/tez/tree/05d7725d
Diff: http://git-wip-us.apache.org/repos/asf/tez/diff/05d7725d

Branch: refs/heads/master
Commit: 05d7725d28de2d9f7166c55d40f6e8e28d8cd3a8
Parents: a2d120b
Author: Hitesh Shah <hi...@apache.org>
Authored: Tue Jun 7 19:50:02 2016 -0700
Committer: Hitesh Shah <hi...@apache.org>
Committed: Tue Jun 7 19:50:02 2016 -0700

----------------------------------------------------------------------
 CHANGES.txt                                                        | 1 +
 .../src/test/java/org/apache/tez/dag/app/dag/impl/TestCommit.java  | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/05d7725d/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 4a5bffe..f31f738 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@ INCOMPATIBLE CHANGES
 
 ALL CHANGES:
 
+  TEZ-2846. Flaky test: TestCommit.testVertexCommit_OnDAGSuccess.
   TEZ-3289. Tez Example MRRSleep job does not set Staging dir correctly on secure cluster.
   TEZ-3276. Tez Example MRRSleep job fails when tez.staging-dir fs is not same as default FS.
   TEZ-3280. LOG MRInputHelpers split generation message as INFO

http://git-wip-us.apache.org/repos/asf/tez/blob/05d7725d/tez-dag/src/test/java/org/apache/tez/dag/app/dag/impl/TestCommit.java
----------------------------------------------------------------------
diff --git a/tez-dag/src/test/java/org/apache/tez/dag/app/dag/impl/TestCommit.java b/tez-dag/src/test/java/org/apache/tez/dag/app/dag/impl/TestCommit.java
index fd56495..c723c9a 100644
--- a/tez-dag/src/test/java/org/apache/tez/dag/app/dag/impl/TestCommit.java
+++ b/tez-dag/src/test/java/org/apache/tez/dag/app/dag/impl/TestCommit.java
@@ -230,7 +230,6 @@ public class TestCommit {
 
     @Override
     public void commitOutput() throws IOException {
-      ++commitCounter;
       while (blockCommit) {
         try {
           Thread.sleep(100);
@@ -239,6 +238,7 @@ public class TestCommit {
           throw new IOException(e);
         }
       }
+      ++commitCounter;
       if (throwError) {
         throw new RuntimeException("I can throwz exceptions in commit");
       }