scionproto / scion

SCION Internet Architecture
https://scion.org
Apache License 2.0
377 stars 158 forks source link

sqlite: Use upsert #3201

Open oncilla opened 4 years ago

oncilla commented 4 years ago

The go sqlite3 driver ships with its own sqlite3 implementation. Luckily, that is 3.24.0 and supports upsert.

We should move the beacondbsqlite and pathdbsqlite to use upsert to avoid one app-db roundtrip

lukedirtwalker commented 4 years ago

Do we already have a new enough version of the lib?

oncilla commented 4 years ago

The vendored library should be new enough, yes

matzf commented 10 months ago

Just as an update: currently there are two different sqlite implementations that we can build against:

So both versions should fully support upsert (last update according to the see docs was in in sqlite 3.35.0).