shiyiky commented on code in PR #3186:
URL: https://github.com/apache/flink-cdc/pull/3186#discussion_r1560513053
##########
docs/content/docs/connectors/mysql.md:
##########
@@ -205,6 +205,16 @@ pipeline:
<td>Integer</td>
<td>The connection pool size.</td>
</tr>
+ <tr>
+ <td>table.exclude.list</td>
Review Comment:
I agree with some of your views, and I think they are very nice. Here, I
will explain why I am doing this:
1.Most users use 'db. *' when using whole database synchronization, but they
also need to exclude some tables that do not need synchronization. If regular
expressions are used to reverse exclude, and regular expressions are not easy
to write, and this is an unclear exclusion, users need to constantly test and
write. If there is an option to exclude tables, 'db.table1, db.table2' is very
obvious and observable.
2.The process of writing and testing regular expressions is tedious if they
have many tables.
3.The 'table. include. list' parameter is the same as the Debezium
parameter, but the principle is different from Debezium. It may be a better
choice to change the parameter name to a different one.
4.We need to hand over simplicity to users and leave complexity to ourselves.
5. I referred to the parameters for CDAS whole database synchronization in
Alibaba Cloud's real-time computing of Flink and the parameters for whole
database synchronization in Flink Doris Connector connector
--
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
