[improve] console sink options (#8743)

Author: liunaijie

seatunnel-ci-tools/src/test/java/org/apache/seatunnel/api/ConnectorOptionCheckTest.java

@@ -208,7 +208,6 @@ private Set<String> buildWhiteList() {
         whiteList.add("SocketSinkOptions");
         whiteList.add("ClickhouseSinkOptions");
         whiteList.add("SelectDBSinkOptions");
-        whiteList.add("ConsoleSinkOptions");
         whiteList.add("PrometheusSinkOptions");
         whiteList.add("FirestoreSinkOptions");
         whiteList.add("ClickhouseSourceOptions");

seatunnel-connectors-v2/connector-console/src/main/java/org/apache/seatunnel/connectors/seatunnel/console/sink/ConsoleSink.java

@@ -31,9 +31,6 @@
 import java.util.List;
 import java.util.Optional;
 
-import static org.apache.seatunnel.connectors.seatunnel.console.sink.ConsoleSinkFactory.LOG_PRINT_DATA;
-import static org.apache.seatunnel.connectors.seatunnel.console.sink.ConsoleSinkFactory.LOG_PRINT_DELAY;
-
 public class ConsoleSink extends AbstractSimpleSink<SeaTunnelRow, Void>
         implements SupportMultiTableSink, SupportSchemaEvolutionSink {
     private final SeaTunnelRowType seaTunnelRowType;
@@ -43,8 +40,8 @@ public class ConsoleSink extends AbstractSimpleSink<SeaTunnelRow, Void>
 
     public ConsoleSink(CatalogTable catalogTable, ReadonlyConfig options) {
         this.catalogTable = catalogTable;
-        this.isPrintData = options.get(LOG_PRINT_DATA);
-        this.delayMs = options.get(LOG_PRINT_DELAY);
+        this.isPrintData = options.get(ConsoleSinkOptions.LOG_PRINT_DATA);
+        this.delayMs = options.get(ConsoleSinkOptions.LOG_PRINT_DELAY);
         this.seaTunnelRowType = catalogTable.getTableSchema().toPhysicalRowDataType();
     }
 

seatunnel-connectors-v2/connector-console/src/main/java/org/apache/seatunnel/connectors/seatunnel/console/sink/ConsoleSinkFactory.java

@@ -17,11 +17,8 @@
 
 package org.apache.seatunnel.connectors.seatunnel.console.sink;
 
-import org.apache.seatunnel.api.configuration.Option;
-import org.apache.seatunnel.api.configuration.Options;
 import org.apache.seatunnel.api.configuration.ReadonlyConfig;
 import org.apache.seatunnel.api.configuration.util.OptionRule;
-import org.apache.seatunnel.api.options.SinkConnectorCommonOptions;
 import org.apache.seatunnel.api.table.connector.TableSink;
 import org.apache.seatunnel.api.table.factory.Factory;
 import org.apache.seatunnel.api.table.factory.TableSinkFactory;
@@ -32,20 +29,6 @@
 @AutoService(Factory.class)
 public class ConsoleSinkFactory implements TableSinkFactory {
 
-    public static final Option<Boolean> LOG_PRINT_DATA =
-            Options.key("log.print.data")
-                    .booleanType()
-                    .defaultValue(true)
-                    .withDescription(
-                            "Flag to determine whether data should be printed in the logs.");
-
-    public static final Option<Integer> LOG_PRINT_DELAY =
-            Options.key("log.print.delay.ms")
-                    .intType()
-                    .defaultValue(0)
-                    .withDescription(
-                            "Delay in milliseconds between printing each data item to the logs.");
-
     @Override
     public String factoryIdentifier() {
         return "Console";
@@ -55,9 +38,9 @@ public String factoryIdentifier() {
     public OptionRule optionRule() {
         return OptionRule.builder()
                 .optional(
-                        LOG_PRINT_DATA,
-                        LOG_PRINT_DELAY,
-                        SinkConnectorCommonOptions.MULTI_TABLE_SINK_REPLICA)
+                        ConsoleSinkOptions.LOG_PRINT_DATA,
+                        ConsoleSinkOptions.LOG_PRINT_DELAY,
+                        ConsoleSinkOptions.MULTI_TABLE_SINK_REPLICA)
                 .build();
     }
 

seatunnel-connectors-v2/connector-console/src/main/java/org/apache/seatunnel/connectors/seatunnel/console/sink/ConsoleSinkOptions.java

@@ -0,0 +1,39 @@
+/*
+ * 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.seatunnel.connectors.seatunnel.console.sink;
+
+import org.apache.seatunnel.api.configuration.Option;
+import org.apache.seatunnel.api.configuration.Options;
+import org.apache.seatunnel.api.options.SinkConnectorCommonOptions;
+
+public class ConsoleSinkOptions extends SinkConnectorCommonOptions {
+
+    public static final Option<Boolean> LOG_PRINT_DATA =
+            Options.key("log.print.data")
+                    .booleanType()
+                    .defaultValue(true)
+                    .withDescription(
+                            "Flag to determine whether data should be printed in the logs.");
+
+    public static final Option<Integer> LOG_PRINT_DELAY =
+            Options.key("log.print.delay.ms")
+                    .intType()
+                    .defaultValue(0)
+                    .withDescription(
+                            "Delay in milliseconds between printing each data item to the logs.");
+}