Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bugFix][Connector-V2][Kafka & Canal] The settings for database.include and table.include are not taking effect. (#6870) #6871

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

KelvinChi
Copy link

@KelvinChi KelvinChi commented May 20, 2024

Purpose of this pull request

This pull request is intended to fix a bug in the Connector-V2 Kafka and Canal modules where data cannot be precisely filtered based on database and table names, meaning the issue of being unable to use database.include and table.include. The specific issue description is written in #6870.

Does this PR introduce any user-facing change?

NO

How was this patch tested?

Package the connector-kafka-2.3.5-2.12.15.jar and place it in the test environment to complete the Kafka data reading test.

Check list

@Hisoka-X Hisoka-X added the First-time contributor First-time contributor label May 20, 2024
@Hisoka-X
Copy link
Member

Hi @KelvinChi , seem like you git history has some problem. You can create a branch not named dev, then push any commit which you want to be merged. Finally create a PR with these branch not dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
First-time contributor First-time contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants