Skip to content

Fix issue 42856 and MaterializedMySQL improvement#42931

Merged
Avogar merged 9 commits intoClickHouse:masterfrom
zzsmdfj:MaterializedMySQL_improvement_bug_fix
Nov 14, 2022
Merged

Fix issue 42856 and MaterializedMySQL improvement#42931
Avogar merged 9 commits intoClickHouse:masterfrom
zzsmdfj:MaterializedMySQL_improvement_bug_fix

Conversation

@zzsmdfj
Copy link
Copy Markdown
Contributor

@zzsmdfj zzsmdfj commented Nov 3, 2022

Changelog category (leave one):

  • Improvement

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

bugfix: #42856 ignore Mysql binlog SAVEPOINT event.

improvement:

  • MaterializedMySQL support 'alter table xx rename xx '
  • MaterializedMySQL support column qualifier: INVISIBLE CHARSET
  • MaterializedMySQL support 'ALTER TABLE xx COMMENT='comment...'

@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Nov 3, 2022
@evillique evillique added the can be tested Allows running workflows for external contributors label Nov 4, 2022
@Avogar Avogar self-assigned this Nov 8, 2022
@Avogar
Copy link
Copy Markdown
Member

Avogar commented Nov 10, 2022

@zzsmdfj please fix style checks

QUERY_EVENT_MULTI_TXN_FLAG = 1,
QUERY_EVENT_XA = 2
QUERY_EVENT_XA = 2,
QUERY_EVENT_OTHER = 3
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you call this QUERY_SAVEPOINT please? We are using that name in our fork (to be merged upstream soon), good to avoid unnecessary naming conflicts :-)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok

@zzsmdfj
Copy link
Copy Markdown
Contributor Author

zzsmdfj commented Nov 11, 2022

@zzsmdfj please fix style checks

done

@Avogar Avogar merged commit eacd7e1 into ClickHouse:master Nov 14, 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-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants