You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flume.apache.org by es...@apache.org on 2011/08/12 02:48:45 UTC

svn commit: r1156915 - in /incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node: NodeManager.java nodemanager/AbstractLogicalNodeManager.java

Author: esammer
Date: Fri Aug 12 00:48:45 2011
New Revision: 1156915

URL: http://svn.apache.org/viewvc?rev=1156915&view=rev
Log:
- Added NodeManager method for getting a node by name. - No longer return a copy of nodes from AbstractNodeManager#getNodes().

Modified:
    incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/NodeManager.java
    incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/nodemanager/AbstractLogicalNodeManager.java

Modified: incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/NodeManager.java
URL: http://svn.apache.org/viewvc/incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/NodeManager.java?rev=1156915&r1=1156914&r2=1156915&view=diff
==============================================================================
--- incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/NodeManager.java (original)
+++ incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/NodeManager.java Fri Aug 12 00:48:45 2011
@@ -15,4 +15,6 @@ public interface NodeManager extends Lif
 
   public void setNodes(Set<LogicalNode> nodes);
 
+  public LogicalNode getNode(String name);
+
 }

Modified: incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/nodemanager/AbstractLogicalNodeManager.java
URL: http://svn.apache.org/viewvc/incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/nodemanager/AbstractLogicalNodeManager.java?rev=1156915&r1=1156914&r2=1156915&view=diff
==============================================================================
--- incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/nodemanager/AbstractLogicalNodeManager.java (original)
+++ incubator/flume/branches/flume-728/flume-ng-node/src/main/java/org/apache/flume/node/nodemanager/AbstractLogicalNodeManager.java Fri Aug 12 00:48:45 2011
@@ -31,8 +31,19 @@ abstract public class AbstractLogicalNod
   }
 
   @Override
+  public LogicalNode getNode(String name) {
+    for (LogicalNode node : getNodes()) {
+      if (node.getName().equals(name)) {
+        return node;
+      }
+    }
+
+    return null;
+  }
+
+  @Override
   public Set<LogicalNode> getNodes() {
-    return new HashSet<LogicalNode>(nodes);
+    return nodes;
   }
 
   @Override