Skip to content

Commit

Permalink
Support for the V ORM
Browse files Browse the repository at this point in the history
DO NOT MERGE.

This is partial support for the ORM. However, there are some challenges
that need to be addressed before this can be properly reviewed and
landed:

1. The ORM in V requires drivers to be hard-coded. See #90.

2. The Connection doesn't _really_ implement orm.Connection because the
vsql Connection is required to be mut and the current interface
definition does not allow this.

3. We need to create a new test suite for the ORM. `vsql/orm_test.v`
filled with combinations of statements "sql" commands will work just
fine. Specifically, we need to test different combinations of
expressions and types.

Fixes #90
  • Loading branch information
elliotchance committed Dec 31, 2023
1 parent 061e53a commit dc6a494
Show file tree
Hide file tree
Showing 14 changed files with 1,462 additions and 54 deletions.
3 changes: 3 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,9 @@ btree-test: oldv
sql-test: oldv
$(V) -stats $(BUILD_OPTIONS) test vsql/sql_test.v

orm-test: oldv
$(V) -stats $(BUILD_OPTIONS) test vsql/orm_test.v

# CLI Tests

cli-test: bin/vsql
Expand Down
1 change: 1 addition & 0 deletions docs/features.rst
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ Features

custom-functions.rst
in-memory-database.rst
orm.rst
prepared-statements.rst
server.rst
transactions.rst
Expand Down

0 comments on commit dc6a494

Please sign in to comment.