You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by je...@apache.org on 2016/10/10 19:16:30 UTC

tez git commit: TEZ-3464. Fix findbugs warnings in tez-dag mainLoop (jeagles)

Repository: tez
Updated Branches:
  refs/heads/master e9fb41864 -> 54fcc0ac4


TEZ-3464. Fix findbugs warnings in tez-dag mainLoop (jeagles)


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

Branch: refs/heads/master
Commit: 54fcc0ac45e667729a3d9d7c936c5dfa1f22b0db
Parents: e9fb418
Author: Jonathan Eagles <je...@yahoo-inc.com>
Authored: Mon Oct 10 14:16:13 2016 -0500
Committer: Jonathan Eagles <je...@yahoo-inc.com>
Committed: Mon Oct 10 14:16:13 2016 -0500

----------------------------------------------------------------------
 CHANGES.txt                  | 3 +++
 tez-dag/findbugs-exclude.xml | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/54fcc0ac/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 10fbd92..d985b9b 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@ INCOMPATIBLE CHANGES
 
 ALL CHANGES:
 
+  TEZ-3464. Fix findbugs warnings in tez-dag mainLoop
   TEZ-3330. Propagate additional config parameters when running MR jobs via Tez.
   TEZ-3335. DAG client thinks app is still running when app status is null
   TEZ-3437. Improve synchronization and the progress report behavior for Inputs from TEZ-3317.
@@ -120,6 +121,7 @@ INCOMPATIBLE CHANGES
 
 ALL CHANGES:
 
+  TEZ-3464. Fix findbugs warnings in tez-dag mainLoop
   TEZ-3330. Propagate additional config parameters when running MR jobs via Tez.
   TEZ-3335. DAG client thinks app is still running when app status is null
   TEZ-3460. Fix precommit release audit warning.
@@ -615,6 +617,7 @@ INCOMPATIBLE CHANGES
 
 ALL CHANGES:
 
+  TEZ-3464. Fix findbugs warnings in tez-dag mainLoop
   TEZ-3335. DAG client thinks app is still running when app status is null
   TEZ-3460. Fix precommit release audit warning.
   TEZ-3368. NPE in DelayedContainerManager

http://git-wip-us.apache.org/repos/asf/tez/blob/54fcc0ac/tez-dag/findbugs-exclude.xml
----------------------------------------------------------------------
diff --git a/tez-dag/findbugs-exclude.xml b/tez-dag/findbugs-exclude.xml
index 5eed7eb..e8adbb3 100644
--- a/tez-dag/findbugs-exclude.xml
+++ b/tez-dag/findbugs-exclude.xml
@@ -220,7 +220,7 @@
   <!-- TEZ-1959 -->
   <Match>
     <Class name="org.apache.tez.dag.app.rm.YarnTaskSchedulerService$DelayedContainerManager"/>
-    <Method name="run"/><Field name="drainedDelayedContainersForTest"/>
+    <Method name="mainLoop"/><Field name="drainedDelayedContainersForTest"/>
     <Bug pattern="JLM_JSR166_UTILCONCURRENT_MONITORENTER"/>
   </Match>