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/07/29 15:20:34 UTC
tez git commit: TEZ-3384. Fix
TestATSV15HistoryLoggingService::testDAGGroupingGroupingEnabled unit test.
(Sushmitha Sreenivasan via hitesh)
Repository: tez
Updated Branches:
refs/heads/master f27f4a13c -> 3546a411a
TEZ-3384. Fix TestATSV15HistoryLoggingService::testDAGGroupingGroupingEnabled unit test. (Sushmitha Sreenivasan via hitesh)
Project: http://git-wip-us.apache.org/repos/asf/tez/repo
Commit: http://git-wip-us.apache.org/repos/asf/tez/commit/3546a411
Tree: http://git-wip-us.apache.org/repos/asf/tez/tree/3546a411
Diff: http://git-wip-us.apache.org/repos/asf/tez/diff/3546a411
Branch: refs/heads/master
Commit: 3546a411ad1e406ec4c603cffe04956f98315011
Parents: f27f4a1
Author: Hitesh Shah <hi...@apache.org>
Authored: Fri Jul 29 08:20:13 2016 -0700
Committer: Hitesh Shah <hi...@apache.org>
Committed: Fri Jul 29 08:20:13 2016 -0700
----------------------------------------------------------------------
CHANGES.txt | 2 ++
.../history/logging/ats/TestATSV15HistoryLoggingService.java | 6 +++---
2 files changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tez/blob/3546a411/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index cdbd4c2..1df4d09 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@ INCOMPATIBLE CHANGES
ALL CHANGES:
+ TEZ-3384. Fix TestATSV15HistoryLoggingService::testDAGGroupingGroupingEnabled unit test.
TEZ-3382. Tez analyzer: Should be resilient to new counters.
TEZ-3379. Tez analyzer: Move sysout to log4j.
TEZ-3376. Fix groupId generation to account for dagId starting with 1.
@@ -93,6 +94,7 @@ INCOMPATIBLE CHANGES
ALL CHANGES:
+ TEZ-3384. Fix TestATSV15HistoryLoggingService::testDAGGroupingGroupingEnabled unit test.
TEZ-3376. Fix groupId generation to account for dagId starting with 1.
TEZ-3359. Add granular log levels for HistoryLoggingService.
TEZ-3374. Change TEZ_HISTORY_LOGGING_TIMELINE_NUM_DAGS_PER_GROUP conf key name.
http://git-wip-us.apache.org/repos/asf/tez/blob/3546a411/tez-plugins/tez-yarn-timeline-history-with-fs/src/test/java/org/apache/tez/dag/history/logging/ats/TestATSV15HistoryLoggingService.java
----------------------------------------------------------------------
diff --git a/tez-plugins/tez-yarn-timeline-history-with-fs/src/test/java/org/apache/tez/dag/history/logging/ats/TestATSV15HistoryLoggingService.java b/tez-plugins/tez-yarn-timeline-history-with-fs/src/test/java/org/apache/tez/dag/history/logging/ats/TestATSV15HistoryLoggingService.java
index 87a48f6..1869b56 100644
--- a/tez-plugins/tez-yarn-timeline-history-with-fs/src/test/java/org/apache/tez/dag/history/logging/ats/TestATSV15HistoryLoggingService.java
+++ b/tez-plugins/tez-yarn-timeline-history-with-fs/src/test/java/org/apache/tez/dag/history/logging/ats/TestATSV15HistoryLoggingService.java
@@ -123,16 +123,16 @@ public class TestATSV15HistoryLoggingService {
public void testDAGGroupingGroupingEnabled() throws Exception {
int numDagsPerGroup = 100;
ATSV15HistoryLoggingService service = createService(numDagsPerGroup);
- TezDAGID dagId1 = TezDAGID.getInstance(appId, 0);
+ TezDAGID dagId1 = TezDAGID.getInstance(appId, 1);
for (DAGHistoryEvent event : makeHistoryEvents(dagId1, service)) {
service.handle(event);
}
- TezDAGID dagId2 = TezDAGID.getInstance(appId, numDagsPerGroup - 1);
+ TezDAGID dagId2 = TezDAGID.getInstance(appId, numDagsPerGroup );
for (DAGHistoryEvent event : makeHistoryEvents(dagId2, service)) {
service.handle(event);
}
- TezDAGID dagId3 = TezDAGID.getInstance(appId, numDagsPerGroup);
+ TezDAGID dagId3 = TezDAGID.getInstance(appId, numDagsPerGroup + 1);
for (DAGHistoryEvent event : makeHistoryEvents(dagId3, service)) {
service.handle(event);
}