Pure-go Sqlite

[aymanbagabas]

Hi all 👋

Couldn't help but notice that Miniflux doesn't support Sqlite because the driver needs CGo and GCC. Would like to share that there is a pure-go Sqlite driver that doesn't need CGo nor GCC https://gitlab.com/cznic/sqlite would be great if Miniflux supports Sqlite as its a lot lighter that Postgres.

[hauleth]

That would also make backing everything up way easier.

[gergan]

Hi @fguillot - are you open for a pull request adding sqlite support?

[fhemberger]

@gergan I took a very quick peek at the code the other day. Seems you'd need to borrow two methods from the pq package, which are used in Miniflux:

• Array
• QuoteIdentifier

Couldn't find any other code specificly tied to Postgres at first glance. So it looks like changing database drivers could be feasible without too much of a hassle.

[fguillot]

Hi @fguillot - are you open for a pull request adding sqlite support?

I'm not against that. However, I'm a little bit concerned about the number of changes required.

Not sure either about the maturity of pure Go Sqlite drivers compared to https://github.com/mattn/go-sqlite3

[gergan]

I've had a quick look at the table definitions (DDL) and there will be some problems - for example the hstore extension, there is no "with timezone" datetime type in sqlite. Probably there will be tons of other problems too. I will not have the time in January to do any work on this.
Probably the best way would be to just try to make the code work with sqlite and if the changes to the code are not minimal, than probably it is not a good idea.

[hauleth]

It is weird that Miniflux uses timestamp with timezone at all, as this type is in general discouraged if you are using any form of ORM or anything. It has only a little bit sense when there is human operator on the DB, but other than that it introduces more problems than it solves. And hstore technically can be replaced by JSON fields.

[E021ntox]

sqlite, please. postgresql is the biggest weakness of miniflux.

[pentago]

Id love sqlite too.

[clarfonthey]

Just poking in here: I know that GoToSocial also had to look for a pure-Go binding to SQLite, and this is the one they use: https://gitlab.com/cznic/sqlite

Probably worth checking out.

[gergan]

There are multiple implementations of Go SQLite without CGo - SQLite Benchmarks. The one that you mean (@clarfonthey) is probably the modernc.org one in the benchmark. I have looked over the database once more yesterday, and the hstore field was converted to separate fields in the meantime. But there are many timestamp with time zone fields there, and I'm not sure why they need to be in local time zones. As @hauleth already said, this probably has some historical reasons - the first step would probably be to try and convert the complete database to UTC. And after that, look at what still needs to be done for SQLite.

[jbrodriguez]

hope this gains some traction, id love to go back to a lightweight version of google reader