-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Rename module and package from MemSQL to SingleStore #11482
Conversation
74dea86
to
9408721
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.
There are some remaining usages:
MEMSQL_* constants in SingleStoreClient.
Some configs in SingleStoreConfig.
Rest of them are in test classes like comments and the DataSetup
implementations.
plugin/trino-singlestore/src/test/java/io/trino/plugin/singlestore/SingleStoreQueryRunner.java
Outdated
Show resolved
Hide resolved
plugin/trino-singlestore/src/main/java/io/trino/plugin/singlestore/SingleStoreConfig.java
Outdated
Show resolved
Hide resolved
9408721
to
a330813
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.
LGTM. We can follow-up on non-user facing stuff later.
a330813
to
f1c1ff0
Compare
I pushed a fix for failed tests, verified everything passes locally. Change is https://github.com/trinodb/trino/pull/11482/files#diff-c8a7eb785c3399d1d0c3a88518918a10c40911ed811c7d2328d9bdffe5a12b62R452-R462 |
f1c1ff0
to
2ad8cf2
Compare
Description
Rename module and package from MemSQL to SingleStore
Documentation
(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.
Release notes
(x) No release notes entries required.
( ) Release notes entries required with the following suggested text: