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:49:03 UTC

svn commit: r1156922 - /incubator/flume/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/LogicalNode.java

Author: esammer
Date: Fri Aug 12 00:49:03 2011
New Revision: 1156922

URL: http://svn.apache.org/viewvc?rev=1156922&view=rev
Log:
- Protect myself from stopping a service that hasn't been started.

Modified:
    incubator/flume/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/LogicalNode.java

Modified: incubator/flume/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/LogicalNode.java
URL: http://svn.apache.org/viewvc/incubator/flume/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/LogicalNode.java?rev=1156922&r1=1156921&r2=1156922&view=diff
==============================================================================
--- incubator/flume/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/LogicalNode.java (original)
+++ incubator/flume/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/LogicalNode.java Fri Aug 12 00:49:03 2011
@@ -110,6 +110,9 @@ public class LogicalNode implements Life
   public void stop(Context context) throws LifecycleException,
       InterruptedException {
 
+    Preconditions.checkState(LifecycleState.START.equals(lifecycleState),
+        "It is illegal to stop something that isn't running");
+
     logger.info("Stopping logical node:{}", this);
 
     driver.setShouldStop(true);