Skip to content

refactor(database) :: route queries through SQLPage-owned driver types - #1596

Open
81reap wants to merge 2 commits into
mainfrom
81reap/sqlx-next/4-driver-seam
Open

81reap wants to merge 2 commits into
mainfrom
81reap/sqlx-next/4-driver-seam

Conversation

@81reap

@81reap 81reap commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

The rest of SQLPage used sqlx-oldapi's AnyPool, PoolConnection, rows and errors directly. The new driver module now owns those types:

Each enum has one variant for now. sqlx 0.9 can now be supported (issue #1056).

Behavior does not change.

Refs #1056


Stack created with GitHub Stacks CLI • Give Feedback 💬

@81reap
81reap added this pull request to stack #1599 November 12, 2026 06:22
@lovasoa

lovasoa commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

I am not a huge fan of maintaining knowledge of both oldapi and newapi in the codebase at once. I welcome all the cleanups, cleaner module separations, etc we can do on main, and I'm ready to merge newapi all at once when it has been sufficiently exercised (maybe try it on some of the open source SQLPage apps out there first to increase confidence)

@lovasoa

lovasoa commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

Another thing we can do on main is improving our end to end coverage in the automatically tested examples. Currently we don't have realistic end to end tests for duckdb or sqlserver, and I think adding at least one (medium complexity) example app for each, exercising each database's specificity, would greatly derisk the migration.

@lovasoa
lovasoa force-pushed the 81reap/sqlx-next/4-driver-seam branch from 1b15aa4 to 32e94f8 Compare November 12, 2026 09:48
@datadog-official

datadog-official Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Pipelines

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: f06e4d6 | Docs | Give us feedback!

@81reap

81reap commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator Author

seems like this is the same thead as this convo so cross linking :: #1597 (comment)

Currently we don't have realistic end to end tests for duckdb or sqlserver, and I think adding at least one (medium complexity) example app for each, exercising each database's specificity, would greatly derisk the migration.

also 100% aligned. once we have a way to be able to run SQLPage against sqlx-oldapi and unstable-sqlx, we can start adding e2e tests to derisk and help build confidence in the Alpha -> Beta dialup

@lovasoa

lovasoa commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

I'd like to first add the tests, then let them run naturally on ci in both branches.

The rest of SQLPage used sqlx-oldapi's AnyPool, PoolConnection, rows and errors directly. The new driver module now owns those types:

Each enum has one variant for now. sqlx 0.9 can now be supported (issue #1056).

Behavior does not change.

Refs #1056
@lovasoa
lovasoa force-pushed the 81reap/sqlx-next/4-driver-seam branch from 32e94f8 to f06e4d6 Compare November 12, 2026 22:07
Base automatically changed from 81reap/sqlx-next/3-row-columns to main November 12, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants