From 943dfc78d17c0df4ad398b9b4747562b9d4a9c84 Mon Sep 17 00:00:00 2001 From: Szilard Nemeth Date: Fri, 9 Aug 2019 09:49:18 +0200 Subject: [PATCH] YARN-9094: Remove unused interface method: NodeResourceUpdaterPlugin#handleUpdatedResourceFromRM. Contributed by Gergely Pollak (cherry picked from commit 72d7e570a73989aa18b737c0e642d570a55c6781) --- .../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 88f77ed12ed..107e43c4ce9 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