Support other database backends #167

New Issue

Jef Roosens · 2022-05-07T11:17:20Z

Jef Roosens commented

2022-05-07 11:17:20 +00:00

Vlib has support for sqlite, postgresql & mysql/mariadb. If we could re-write the code to choose which database type to use, the ORM would allow us to choose one over the other.

Jef Roosens added the

enhancement

label 2022-05-07 11:17:20 +00:00

Jef Roosens commented

2022-05-28 14:03:43 +00:00

Poster

The ORM is currently not complete enough to allow for this.

Jef Roosens commented

2022-06-28 09:02:26 +00:00

Poster

I think this could be accomplished by not using the ORM syntax at all. Instead, all database operations could rely on the orm.Connection interface. Then the type of the database connection could be just orm.Connection, allowing multiple database backends!

Of course, this still leaves the problem of migrations. We'd have to write migrations for every database backend that we wish to support.

I think this could be accomplished by not using the ORM syntax at all. Instead, all database operations could rely on the `orm.Connection` interface. Then the type of the database connection could be just `orm.Connection`, allowing multiple database backends! Of course, this still leaves the problem of migrations. We'd have to write migrations for every database backend that we wish to support.

Jef Roosens commented

2022-10-29 12:32:53 +00:00

Poster

this PR could make this simpler, as it would allow us to use a generic orm.Connection object for ORM operations, leaving only the problem of custom migrations.

[this PR](https://github.com/vlang/v/pull/16076) could make this simpler, as it would allow us to use a generic `orm.Connection` object for ORM operations, leaving only the problem of custom migrations.

Jef Roosens added a new dependency 2022-11-02 07:21:51 +00:00

#283 Fully database-agnostic code

Jef Roosens commented

2022-11-02 07:22:53 +00:00

Poster

The above PR has not been merged in the V 0.3.2 release, so we'll most likely have to wait until V 0.3.3 to fully implement this. However, we can already pave the way by implementing #283.

Jef Roosens added the

V

label 2022-11-02 07:34:17 +00:00

Jef Roosens commented

2023-01-31 09:03:55 +00:00

Poster

The PR didn't make it into V 0.3.3 🤷‍♂️

Sign in to join this conversation.