-
Notifications
You must be signed in to change notification settings - Fork 29
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
I hardcoded it into the base `DbApi`/`Query` APIs, rather than as a dialect. Sqlite doesn't support it, but it is a builtin part of the JDBC interface, so it's a bit on the fence which side it should go. But the fact that it does need special support in the core `DbApi` implementation makes me lean towards treating it as a builtin We need to pass an explicit `[R]` type parameter to `.getGeneratedKeys` because our `Table` model is not rich enough to tell us what the auto generated primary keys of the table are. This is in contrast to richer models like SLICK which do contain this information. We can streamline this later if necessary, but passing in `[Int]` or whatever is not a huge hardship.
- Loading branch information
Showing
11 changed files
with
700 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.