/*
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you 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 org.apache.flink.runtime.clusterframework.overlays;

import org.apache.flink.configuration.Configuration;
import org.apache.flink.core.fs.Path;
import org.apache.flink.core.plugin.PluginConfig;
import org.apache.flink.runtime.clusterframework.ContainerSpecification;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import javax.annotation.Nullable;

import java.io.File;
import java.io.IOException;

import static org.apache.flink.configuration.ConfigConstants.ENV_FLINK_BIN_DIR;
import static org.apache.flink.configuration.ConfigConstants.ENV_FLINK_CONF_DIR;
import static org.apache.flink.configuration.ConfigConstants.ENV_FLINK_HOME_DIR;
import static org.apache.flink.configuration.ConfigConstants.ENV_FLINK_LIB_DIR;
import static org.apache.flink.util.Preconditions.checkNotNull;
import static org.apache.flink.util.Preconditions.checkState;

/**
 * Overlays Flink into a container, based on supplied bin/conf/lib directories.
 *
 * The overlayed Flink is indistinguishable from (and interchangeable with)
 * a normal installation of Flink.  For a docker image-based container, it should be
 * possible to bypass this overlay and rely on the normal installation method.
 *
 * The following files are copied to the container:
 *  - flink/bin/
 *  - flink/conf/
 *  - flink/lib/
 */
public class FlinkDistributionOverlay extends AbstractContainerOverlay {

	private static final Logger LOG = LoggerFactory.getLogger(FlinkDistributionOverlay.class);

	static final Path TARGET_ROOT = new Path("flink");

	final File flinkBinPath;
	final File flinkConfPath;
	final File flinkLibPath;
	@Nullable
	final File flinkPluginsPath;

	FlinkDistributionOverlay(File flinkBinPath, File flinkConfPath, File flinkLibPath, @Nullable File flinkPluginsPath) {
		this.flinkBinPath = checkNotNull(flinkBinPath);
		this.flinkConfPath = checkNotNull(flinkConfPath);
		this.flinkLibPath = checkNotNull(flinkLibPath);
		this.flinkPluginsPath = flinkPluginsPath;
	}

	@Override
	public void configure(ContainerSpecification container) throws IOException {

		container.getEnvironmentVariables().put(ENV_FLINK_HOME_DIR, TARGET_ROOT.toString());

		// add the paths to the container specification.
		addPathRecursively(flinkBinPath, TARGET_ROOT, container);
		addPathRecursively(flinkConfPath, TARGET_ROOT, container);
		addPathRecursively(flinkLibPath, TARGET_ROOT, container);
		if (flinkPluginsPath != null) {
			addPathRecursively(flinkPluginsPath, TARGET_ROOT, container);
		}
	}

	public static Builder newBuilder() {
		return new Builder();
	}

	/**
	 * A builder for the {@link FlinkDistributionOverlay}.
	 */
	public static class Builder {
		File flinkBinPath;
		File flinkConfPath;
		File flinkLibPath;
		@Nullable
		File flinkPluginsPath;

		/**
		 * Configures the overlay using the current environment.
		 *
		 * Locates Flink using FLINK_???_DIR environment variables as provided to all Flink processes by config.sh.
		 *
		 * @param globalConfiguration the current configuration.
		 */
		public Builder fromEnvironment(Configuration globalConfiguration) {
			flinkBinPath = getObligatoryFileFromEnvironment(ENV_FLINK_BIN_DIR);
			flinkConfPath = getObligatoryFileFromEnvironment(ENV_FLINK_CONF_DIR);
			flinkLibPath = getObligatoryFileFromEnvironment(ENV_FLINK_LIB_DIR);
			flinkPluginsPath = PluginConfig.getPluginsDir().orElse(null);

			return this;
		}

		public FlinkDistributionOverlay build() {
			return new FlinkDistributionOverlay(flinkBinPath, flinkConfPath, flinkLibPath, flinkPluginsPath);
		}

		private static File getObligatoryFileFromEnvironment(String envVariableName) {
			String directory = System.getenv(envVariableName);
			checkState(directory != null, "the %s environment variable must be set", envVariableName);
			return new File(directory);
		}
	}
}

@@ -1080,66 +1080,8 @@
 n;%0A%0A
-import org.slf4j.Logger;%0Aimport org.slf4j.LoggerFactory;%0A%0A
 impo
@@ -1696,24 +1696,27 @@
 ries.%0A *%0A * 
+%3Cp%3E
 The overlaye
@@ -1946,16 +1946,19 @@
 .%0A *%0A * 
+%3Cp%3E
 The foll
@@ -2132,101 +2132,8 @@
  %7B%0A%0A
-%09private static final Logger LOG = LoggerFactory.getLogger(FlinkDistributionOverlay.class);%0A%0A
 %09sta
@@ -2182,16 +2182,24 @@
 nk%22);%0A%0A%09
+private 
 final Fi
@@ -2212,24 +2212,32 @@
 nkBinPath;%0A%09
+private 
 final File f
@@ -2251,16 +2251,24 @@
 fPath;%0A%09
+private 
 final Fi
@@ -2296,16 +2296,24 @@
 llable%0A%09
+private 
 final Fi
@@ -3510,16 +3510,19 @@
  *%0A%09%09 * 
+%3Cp%3E
 Locates 

Fix checkstyle violations in FlinkDistributionOverlay