-
Notifications
You must be signed in to change notification settings - Fork 466
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
tests: create table from source: platform checks and testdrive #29362
tests: create table from source: platform checks and testdrive #29362
Conversation
962966a
to
3b0c279
Compare
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.
Any other tests planned? I would have added it in parallel-workload too, or are you already working on that?
Absolutely. I am currently about to migrate all existing tests (and additionally keep the original CDC tests). |
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.
nice!
Testdrive(dedent(s)) | ||
for s in [ | ||
f""" | ||
> CREATE TABLE mysql_table_1 FROM SOURCE mysql_source_{self.suffix} (REFERENCE "public_{self.suffix}"."mysql_source_table_1"); |
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.
we could add a text columns option here if we made one of the table columns an enum
value or year
value which are the 2 types that are unsupported natively and we've added text-columns support for: https://materialize.com/docs/sql/create-source/mysql/#supported-types
3b0c279
to
3e3b9b7
Compare
Part of #29510.
Nightly
https://buildkite.com/materialize/nightly/builds?branch=nrainer-materialize%3Atests%2Fcreate-table-from-source