/*
 * Copyright 2013 Yash Parghi
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *   http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package harp.bubble;

import harp.resource.Resource;
import java.io.IOException;
import java.nio.file.Path;

/**
 * An ExecutionBubble is an isolated environment in which a single
 * {@link harp.executable.Executable} is run.
 *
 * <p>Roughly speaking, an {@code ExecutionBubble} corresponds to a temporary directory.
 */
// TODO Things I want from execution bubbles:
// - ability to specify a location/mount point
// - user to run as
// - logs and artifacts storage location
// - environment variables and other harp-specific context variables
// - kill all processes on completion (i.e. in cleanUp)
public interface ExecutionBubble {

  /**
   * Returns the file system location of this bubble, i.e. the temporary directory to which it
   * corresponds.
   */
  Path getLocation();

  /**
   * Clean up all resources used by this bubble.
   */
  void cleanUp() throws IOException;

  /**
   * Add a {@link harp.resource.Resource} to this bubble. Any resources must be added
   * <em>before</em> an {@code Executable} is run in this bubble, otherwise an
   * {@link IllegalStateException} is thrown.
   *
   * @throw IllegalStateException if an {@code Executable} has already been run in this bubble
   */
  void addResource(Resource resource);
}

@@ -1704,16 +1704,17 @@
 * @throw
+s
  Illegal

Fix javadoc typo.