You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by sn...@apache.org on 2019/08/09 07:49:45 UTC

[hadoop] branch trunk updated: YARN-9094: Remove unused interface method: NodeResourceUpdaterPlugin#handleUpdatedResourceFromRM. Contributed by Gergely Pollak

This is an automated email from the ASF dual-hosted git repository.

snemeth pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 72d7e57  YARN-9094: Remove unused interface method: NodeResourceUpdaterPlugin#handleUpdatedResourceFromRM. Contributed by Gergely Pollak
72d7e57 is described below

commit 72d7e570a73989aa18b737c0e642d570a55c6781
Author: Szilard Nemeth <sn...@apache.org>
AuthorDate: Fri Aug 9 09:49:18 2019 +0200

    YARN-9094: Remove unused interface method: NodeResourceUpdaterPlugin#handleUpdatedResourceFromRM. Contributed by Gergely Pollak
---
 .../resourceplugin/NodeResourceUpdaterPlugin.java        | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)

diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/resourceplugin/NodeResourceUpdaterPlugin.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/resourceplugin/NodeResourceUpdaterPlugin.java
index 88f77ed..107e43c 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/resourceplugin/NodeResourceUpdaterPlugin.java
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/resourceplugin/NodeResourceUpdaterPlugin.java
@@ -28,25 +28,11 @@ import org.apache.hadoop.yarn.exceptions.YarnException;
 public abstract class NodeResourceUpdaterPlugin {
   /**
    * Update configured resource for the given component.
-   * @param res resource passed in by external mododule (such as
+   * @param res resource passed in by external module (such as
    *            {@link org.apache.hadoop.yarn.server.nodemanager.NodeStatusUpdater}
    * @throws YarnException when any issue happens.
    */
   public abstract void updateConfiguredResource(Resource res)
       throws YarnException;
 
-  /**
-   * This method will be called when the node's resource is loaded from
-   * dynamic-resources.xml in ResourceManager.
-   *
-   * @param newResource newResource reported by RM
-   * @throws YarnException when any mismatch between NM/RM
-   */
-  public void handleUpdatedResourceFromRM(Resource newResource) throws
-      YarnException {
-    // by default do nothing, subclass should implement this method when any
-    // special activities required upon new resource reported by RM.
-  }
-
-  // TODO: add implementation to update node attribute once YARN-3409 merged.
 }
\ No newline at end of file


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org