You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by rb...@apache.org on 2016/07/04 06:27:34 UTC
tez git commit: TEZ-3313. ATSFileParser : Wrong args passed in
VersionInfo (rbalamohan)
Repository: tez
Updated Branches:
refs/heads/master d7b6eb5b1 -> 57e017357
TEZ-3313. ATSFileParser : Wrong args passed in VersionInfo (rbalamohan)
Project: http://git-wip-us.apache.org/repos/asf/tez/repo
Commit: http://git-wip-us.apache.org/repos/asf/tez/commit/57e01735
Tree: http://git-wip-us.apache.org/repos/asf/tez/tree/57e01735
Diff: http://git-wip-us.apache.org/repos/asf/tez/diff/57e01735
Branch: refs/heads/master
Commit: 57e01735723fd86679f2859b74e6d96da3407c4b
Parents: d7b6eb5
Author: Rajesh Balamohan <rb...@apache.org>
Authored: Mon Jul 4 11:56:47 2016 +0530
Committer: Rajesh Balamohan <rb...@apache.org>
Committed: Mon Jul 4 11:56:47 2016 +0530
----------------------------------------------------------------------
CHANGES.txt | 1 +
.../main/java/org/apache/tez/history/parser/ATSFileParser.java | 2 +-
.../org/apache/tez/history/parser/datamodel/VersionInfo.java | 6 +++---
3 files changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tez/blob/57e01735/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 618d0fb..b58c706 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@ INCOMPATIBLE CHANGES
ALL CHANGES:
+ TEZ-3313. ATSFileParser : Wrong args passed in VersionInfo.
TEZ-3286. Allow clients to set processor reserved memory per vertex (instead of per container).
TEZ-3293. Fetch failures can cause a shuffle hang waiting for memory merge that never starts.
TEZ-3314. Double counting input bytes in MultiMRInput.
http://git-wip-us.apache.org/repos/asf/tez/blob/57e01735/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/ATSFileParser.java
----------------------------------------------------------------------
diff --git a/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/ATSFileParser.java b/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/ATSFileParser.java
index aae20eb..a789536 100644
--- a/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/ATSFileParser.java
+++ b/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/ATSFileParser.java
@@ -156,7 +156,7 @@ public class ATSFileParser extends BaseParser implements ATSData {
String version = tezVersion.optString(Constants.VERSION);
String buildTime = tezVersion.optString(Constants.BUILD_TIME);
String revision = tezVersion.optString(Constants.REVISION);
- this.versionInfo = new VersionInfo(version, buildTime, revision);
+ this.versionInfo = new VersionInfo(version, revision, buildTime);
}
//TODO: might need to parse config info? (e.g, hive settings etc. could consume memory)
}
http://git-wip-us.apache.org/repos/asf/tez/blob/57e01735/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/datamodel/VersionInfo.java
----------------------------------------------------------------------
diff --git a/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/datamodel/VersionInfo.java b/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/datamodel/VersionInfo.java
index 97d18cd..861e50f 100644
--- a/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/datamodel/VersionInfo.java
+++ b/tez-plugins/tez-history-parser/src/main/java/org/apache/tez/history/parser/datamodel/VersionInfo.java
@@ -24,10 +24,10 @@ public class VersionInfo {
private final String revision;
private final String version;
- public VersionInfo(String buildTime, String revision, String version) {
- this.buildTime = buildTime;
- this.revision = revision;
+ public VersionInfo(String version, String revision, String buildTime) {
this.version = version;
+ this.revision = revision;
+ this.buildTime = buildTime;
}
public String getBuildTime() {