-
Notifications
You must be signed in to change notification settings - Fork 0
Renamed connector #23
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
Conversation
| ## Compatibility | ||
|
|
||
| Connector is compatible with R2DBC 1.0.0 spec. |
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.
Maybe we should leave this part here to indicate R2DBC version?
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.
This compatibility is already mentioned above
Driver follow [R2DBC 1.0.0 specifications](https://r2dbc.io/spec/1.0.0.RELEASE/spec/html/)
| @@ -1,3 +1,3 @@ | |||
| // SPDX-License-Identifier: Apache-2.0 | |||
| // Copyright (c) 2020-2024 MariaDB Corporation Ab | |||
| // Copyright (c) 2025-2025 SingleStore, Inc. | |||
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 should leave MariaDB copyright in place and append ours on a new line
| @@ -1,223 +0,0 @@ | |||
| # Change Log | |||
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.
I'd rather leave this file in place, possible with a line like "Here is the first release".
README.md
Outdated
| SingleStore client, 100% Java, compatible with Java8+, apache 2.0 licensed. | ||
| Driver permits PAM authentication that comes with SingleStore. | ||
|
|
||
| Driver follow [R2DBC 1.0.0 specifications](https://r2dbc.io/spec/1.0.0.RELEASE/spec/html/) |
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.
nit: Driver follows
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.
Updated
No description provided.