leonardBang commented on code in PR #3560:
URL: https://github.com/apache/flink-cdc/pull/3560#discussion_r1724885602


##########
flink-cdc-connect/flink-cdc-pipeline-connectors/flink-cdc-pipeline-connector-mysql/src/main/java/org/apache/flink/cdc/connectors/mysql/source/MySqlDataSourceOptions.java:
##########
@@ -261,4 +261,12 @@ public class MySqlDataSourceOptions {
                                     + "If there is a need to use a dot (.) in 
a regular expression to match any character, "
                                     + "it is necessary to escape the dot with 
a backslash."
                                     + "eg. db0.\\.*, db1.user_table_[0-9]+, 
db[1-2].[app|web]_order_\\.*");
+
+    @Experimental
+    public static final ConfigOption<Boolean> 
SCAN_INCREMENTAL_NEWLY_ADDED_TABLE_ENABLED =
+            ConfigOptions.key("scan.incremental.newly-added-table.enabled")
+                    .booleanType()
+                    .defaultValue(false)
+                    .withDescription(
+                            "Whether to scan the ddl and dml statements of 
newly added tables or not in incremental reading stage, by default is false.");

Review Comment:
   We should explain the option works phase and the behavior, and also the diff 
with SCAN_NEWLY_ADDED_TABLE_ENABLED  : do re-snapshot & binlog-reading for 
newly added table when restore
   SCAN_BINLOG_NEWLY_ADDED_TABLE_ENABLED : only do binlog-reading for newly 
added table  during binlog reading phase



##########
flink-cdc-connect/flink-cdc-pipeline-connectors/flink-cdc-pipeline-connector-mysql/src/main/java/org/apache/flink/cdc/connectors/mysql/source/MySqlDataSourceOptions.java:
##########
@@ -261,4 +261,12 @@ public class MySqlDataSourceOptions {
                                     + "If there is a need to use a dot (.) in 
a regular expression to match any character, "
                                     + "it is necessary to escape the dot with 
a backslash."
                                     + "eg. db0.\\.*, db1.user_table_[0-9]+, 
db[1-2].[app|web]_order_\\.*");
+
+    @Experimental
+    public static final ConfigOption<Boolean> 
SCAN_INCREMENTAL_NEWLY_ADDED_TABLE_ENABLED =
+            ConfigOptions.key("scan.incremental.newly-added-table.enabled")

Review Comment:
   scan.binlog.newly-added-table.enabled



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to