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
