Add Configuration Validation

Author: Samrat002

flink-connector-aws/flink-connector-redshift/src/main/java/org/apache/flink/connector/redshift/mode/SinkMode.java

@@ -0,0 +1,25 @@
+/*
+ * 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.connector.redshift.mode;
+
+/** Enum to Define different Modes of Sink supported in Redshift. */
+public enum SinkMode {
+    JDBC,
+    COPY
+}

flink-connector-aws/flink-connector-redshift/src/main/java/org/apache/flink/connector/redshift/sink/config/RedshiftSinkConfigConstants.java

@@ -20,6 +20,7 @@
 
 import org.apache.flink.configuration.ConfigOption;
 import org.apache.flink.configuration.ConfigOptions;
+import org.apache.flink.connector.redshift.mode.SinkMode;
 
 import java.time.Duration;
 
@@ -28,12 +29,6 @@
 /** Constants to be used with the RedshiftSink. */
 public class RedshiftSinkConfigConstants {
 
-    /** Enum to Define different Modes of Sink supported in Redshift. */
-    public enum SinkMode {
-        JDBC,
-        COPY
-    }
-
     public static final ConfigOption<String> HOSTNAME =
             ConfigOptions.key("hostname")
                     .stringType()

flink-connector-aws/flink-connector-redshift/src/main/java/org/apache/flink/connector/redshift/sink/config/RedshiftSinkConfigUtil.java

@@ -19,12 +19,39 @@
 package org.apache.flink.connector.redshift.sink.config;
 
 import org.apache.flink.configuration.Configuration;
+import org.apache.flink.util.FlinkRuntimeException;
 
 /** Utility functions to use with {@link RedshiftSinkConfigConstants}. */
 public class RedshiftSinkConfigUtil {
 
     // private constructor to prevent initialization of utility class.
     private RedshiftSinkConfigUtil() {}
 
-    public static void validateConfigs(final Configuration sinkConfig) {}
+    public static void validateConfigs(final Configuration sinkConfig)
+            throws FlinkRuntimeException {
+        switch (sinkConfig.get(RedshiftSinkConfigConstants.SINK_MODE).toString()) {
+            case "JDBC":
+                validateJdbcAssociatedConfigs(sinkConfig);
+                break;
+            case "COPY":
+                validateCopyAssociatedConfigs(sinkConfig);
+                break;
+            default:
+                throw new FlinkRuntimeException("Invalid Sink Mode");
+        }
+    }
+
+    private static void validateJdbcAssociatedConfigs(Configuration sinkConfig) {
+        if (sinkConfig.getString(RedshiftSinkConfigConstants.HOSTNAME).trim().length() < 3
+                || sinkConfig.getInteger(RedshiftSinkConfigConstants.PORT) < 0) {
+            throw new FlinkRuntimeException("Invalid configuration");
+        }
+    }
+
+    private static void validateCopyAssociatedConfigs(Configuration sinkConfig) {
+        if (sinkConfig.getString(RedshiftSinkConfigConstants.S3_URI) == null
+                || sinkConfig.getString(RedshiftSinkConfigConstants.IAM_ROLE_ARN) == null) {
+            throw new FlinkRuntimeException("Invalid Configuration");
+        }
+    }
 }