Make query builder generalize past &mut Store #224

Open
opened 2020-08-06 16:56:55 +00:00 by gburd · 0 comments
gburd commented 2020-08-06 16:56:55 +00:00 (Migrated from github.com)

Right now, the query builder exposes a simple interface that accepts &mut Store. It's pretty restrictive, since Store is a simple type and since taking a mutable reference enforces linear ordering.

This ticket tracks allowing callers to provide something that can yield the pair of Conn and SQLite connection, so that callers that care about their own connection management can take care of things on their own.

As an aside, I'm surprised that this is as restrictive as I am finding it while writing tests!

Right now, the query builder exposes a simple interface that accepts `&mut Store`. It's pretty restrictive, since `Store` is a simple type and since taking a mutable reference enforces linear ordering. This ticket tracks allowing callers to provide something that can yield the pair of `Conn` and SQLite connection, so that callers that care about their own connection management can take care of things on their own. As an aside, I'm surprised that this is as restrictive as I am finding it while writing tests!
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: greg/mentat#224
No description provided.