Skip to content

Ignore SAVEPOINT queries in MaterializedMySQL#43086

Merged
Avogar merged 1 commit intoClickHouse:masterfrom
stigsb:savepoint-fix
Nov 11, 2022
Merged

Ignore SAVEPOINT queries in MaterializedMySQL#43086
Avogar merged 1 commit intoClickHouse:masterfrom
stigsb:savepoint-fix

Conversation

@stigsb
Copy link
Copy Markdown
Contributor

@stigsb stigsb commented Nov 9, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Handle (ignore) SAVEPOINT queries in MaterializedMySQL

Fixes #42856

@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-bugfix Pull request with bugfix, not backported by default label Nov 9, 2022
@Avogar Avogar self-assigned this Nov 9, 2022
@Avogar Avogar added the can be tested Allows running workflows for external contributors label Nov 9, 2022
@Avogar
Copy link
Copy Markdown
Member

Avogar commented Nov 10, 2022

Seems like it's already fixed in #42931

@stigsb
Copy link
Copy Markdown
Contributor Author

stigsb commented Nov 10, 2022

#42931 does not include tests. I would recommend merging this first, and have #42931 rebase on top (they fix some other things too).

@stigsb stigsb force-pushed the savepoint-fix branch 3 times, most recently from 726c0d4 to 40f269a Compare November 10, 2022 13:41
@Avogar Avogar merged commit a42daf5 into ClickHouse:master Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

can be tested Allows running workflows for external contributors pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MaterializedMySQL got error Unsupported query event:SAVEPOINT __EFSavePoint

3 participants