-
Notifications
You must be signed in to change notification settings - Fork 142
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
Fix mysql advanced sync rule execute error #3044
Conversation
💚 CLA has been signed |
buildkite test this |
buildkite test this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
💔 Failed to create backport PR(s)The backport operation could not be completed due to the following error: The backport PRs will be merged automatically after passing CI. To backport manually run: |
Thanks for the contribution! |
BUG: When configuring the advanced sync rule for the MySQL connector, the sync action throws an error: 'MySQLClient.yield_rows_for_query() missing 1 required positional argument: 'primary_key_columns''.
Root Cause: The yield_rows_for_query method is called without the required primary_key_columns parameter.
Fix: Add the missing primary_key_columns parameter when calling the yield_rows_for_query method.