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 <[email protected]> Authored: Mon Jul 4 11:56:47 2016 +0530 Committer: Rajesh Balamohan <[email protected]> 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() {
