Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ENH: Proper DB connection handling via cmdline arg #1

Open
jerdra opened this issue Dec 7, 2021 · 0 comments
Open

ENH: Proper DB connection handling via cmdline arg #1

jerdra opened this issue Dec 7, 2021 · 0 comments
Labels

Comments

@jerdra
Copy link
Owner

jerdra commented Dec 7, 2021

The rater is still in the development sphere where it creates a .db file in the current directory - this is obviously not ideal.

Instead there should be a cmdline argument where the user may provide authentication and/or an endpoint for where the database should be created. Alternatively, Niviz may write to an existing database instead.

The feature requirements are as follows:

  1. User may provide a local path, in which case a .db file will be outputted using SQLite, this is meant for local sessions only
  2. A user may provide a database connection string, in which case we should require auth info to establish a connection. Niviz can look for an existing table in that database or it should create it's own if none exist
  3. A user may choose not to save a persistent database, in which case an in-memory DB should be used (sqlite3)
@jerdra jerdra added enhancement New feature or request impact: large backend labels Dec 7, 2021
@jerdra jerdra changed the title ENH: Add argument to specify where to write SQL database ENH: Proper DB connection handling via cmdline arg Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant