search

cboettig / neonstore

:package: A local content-based storage system for NEON data
https://cboettig.github.io/neonstore
Other
8 stars 5 forks source link

consider using read-only connection for omit_imported in neon_store #30

Closed cboettig closed 3 years ago

cboettig commented 3 years ago

neon_store should only create a write-able connection when it is sure that we will need to write new data to the database. This way, concurrent calls to are less likely to hit collisions.

https://github.com/cboettig/neonstore/blob/master/R/neon_store.R#L51

cboettig commented 3 years ago

not practical with pluggable backend, since neon_store() call creates a write-access connection by default now. neon_store() should not be called on a workflow that cannot ensure it has write-access to the database.