-
Notifications
You must be signed in to change notification settings - Fork 228
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
merge libsql #1716
merge libsql #1716
Conversation
Add libsql package and initialize it to make it available for sql.Open Signed-off-by: Piotr Jastrzebski <piotr@chiselstrike.com>
Signed-off-by: Piotr Jastrzebski <piotr@chiselstrike.com>
Add libsql package and initialize it to make it available for sql.Open Signed-off-by: Piotr Jastrzebski <piotr@chiselstrike.com>
Signed-off-by: Piotr Jastrzebski <piotr@chiselstrike.com>
3ce9476
to
b1535aa
Compare
sql/sqlite/driver.go
Outdated
sqlclient.DriverOpener(Open), | ||
sqlclient.RegisterTxOpener(OpenTx), | ||
sqlclient.RegisterCodec(MarshalHCL, EvalHCL), | ||
sqlclient.RegisterFlavours("libsql+wss", "libsql+https"), |
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.
At the moment +https won't work as it does not support interactive transactions and is missing some features. We're working on making it better but maybe it's worth to keep https out of scope for this PR?
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've tested the PR against Turso (managed Libsql) and updated https://github.com/chiselstrike/atlas-integration-ci to use the haaawk-libsql
branch for testing. All tests are passing.
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.
No description provided.