review YARN
Project: http://git-wip-us.apache.org/repos/asf/incubator-ignite/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ignite/commit/f6279767 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ignite/tree/f6279767 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ignite/diff/f6279767 Branch: refs/heads/ignite-1.3 Commit: f6279767ad499ca2540f493b93427f44c98d699d Parents: c392cdc Author: Yakov Zhdanov <yzhda...@gridgain.com> Authored: Thu Jul 9 16:22:31 2015 +0300 Committer: Yakov Zhdanov <yzhda...@gridgain.com> Committed: Thu Jul 9 16:22:31 2015 +0300 ---------------------------------------------------------------------- RELEASE_NOTES.txt | 1 + .../src/main/java/org/apache/ignite/yarn/ApplicationMaster.java | 4 +--- .../src/main/java/org/apache/ignite/yarn/IgniteContainer.java | 5 +---- 3 files changed, 3 insertions(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/f6279767/RELEASE_NOTES.txt ---------------------------------------------------------------------- diff --git a/RELEASE_NOTES.txt b/RELEASE_NOTES.txt index 0e22f1f..f68a339 100644 --- a/RELEASE_NOTES.txt +++ b/RELEASE_NOTES.txt @@ -5,6 +5,7 @@ Apache Ignite In-Memory Data Fabric 1.3 --------------------------------------- * Added auto-retries for cache operations in recoverable cases. +* Added integration with Apache YARN. * Fixed several issues with JTA integration. * Fixed several issues with Hibernate L2 cache. * Fixed issue with GAR files in source release. http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/f6279767/modules/yarn/src/main/java/org/apache/ignite/yarn/ApplicationMaster.java ---------------------------------------------------------------------- diff --git a/modules/yarn/src/main/java/org/apache/ignite/yarn/ApplicationMaster.java b/modules/yarn/src/main/java/org/apache/ignite/yarn/ApplicationMaster.java index f57df92..9169178 100644 --- a/modules/yarn/src/main/java/org/apache/ignite/yarn/ApplicationMaster.java +++ b/modules/yarn/src/main/java/org/apache/ignite/yarn/ApplicationMaster.java @@ -74,7 +74,7 @@ public class ApplicationMaster implements AMRMClientAsync.CallbackHandler { private Map<ContainerId, IgniteContainer> containers = new ConcurrentHashMap<>(); /** - * Constructor. + * @param */ public ApplicationMaster(String ignitePath, ClusterProperties props) throws Exception { this.conf = new YarnConfiguration(); @@ -145,8 +145,6 @@ public class ApplicationMaster implements AMRMClientAsync.CallbackHandler { } /** - * Checks that container - * * @param cont Container. * @return {@code True} if */ http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/f6279767/modules/yarn/src/main/java/org/apache/ignite/yarn/IgniteContainer.java ---------------------------------------------------------------------- diff --git a/modules/yarn/src/main/java/org/apache/ignite/yarn/IgniteContainer.java b/modules/yarn/src/main/java/org/apache/ignite/yarn/IgniteContainer.java index a8b0342..313e9e1 100644 --- a/modules/yarn/src/main/java/org/apache/ignite/yarn/IgniteContainer.java +++ b/modules/yarn/src/main/java/org/apache/ignite/yarn/IgniteContainer.java @@ -81,9 +81,6 @@ public class IgniteContainer { * {@inheritDoc} */ @Override public String toString() { - return "IgniteTask " + - "host: [" + nodeId.getHost() + ']' + - ", cpuCores: [" + cpuCores + "]" + - ", mem: [" + mem + "]"; + return "IgniteTask [host=" + nodeId.getHost() + ", cpuCores=" + cpuCores + ", mem=" + mem + ']'; } }
