Skip to content

Conversation

@GOODBOY008
Copy link
Member

Close #2439 .

@GOODBOY008
Copy link
Member Author

@Jiabao-Sun @ruanhang1993 @edmond-kk PTAL.

Copy link
Contributor

@ruanhang1993 ruanhang1993 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @GOODBOY008.
Maybe we should use

.toQuotedString('`')

instead of toString().

Copy link
Contributor

@ruanhang1993 ruanhang1993 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add some tests to cover this bug. Thanks.

Copy link
Contributor

@ruanhang1993 ruanhang1993 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@ruanhang1993 ruanhang1993 merged commit 7801ac9 into apache:master Sep 7, 2023
e-mhui pushed a commit to e-mhui/flink-cdc-connectors that referenced this pull request Oct 18, 2023
GOODBOY008 added a commit to GOODBOY008/flink-cdc that referenced this pull request Oct 30, 2023
GOODBOY008 added a commit to GOODBOY008/flink-cdc that referenced this pull request Oct 30, 2023
GOODBOY008 added a commit to GOODBOY008/flink-cdc that referenced this pull request Oct 30, 2023
leonardBang pushed a commit that referenced this pull request Oct 31, 2023
zhongqishang pushed a commit to zhongqishang/flink-cdc that referenced this pull request Dec 7, 2023
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] sqlservercdc throw a NullPointerException when table name contains point

2 participants