Re: [PATCH] ob-sqlite: Use a transient in-memory database by default

[Max Nikulin]

On 03/05/2023 22:12, Max Nikulin wrote:
> On 03/05/2023 19:59, Rudolf Adamkovič wrote:
> > +++ b/lisp/ob-sqlite.el
...
> > -    (db (cdr (assq :db params)))
> > +    (db (or (cdr (assq :db params)) ":memory:"))
>
> I am unsure what is better, to add a fallback here or to add :db 
> ":memory" to the org-babel-default-header-args:sqlite variable.

Perhaps it is better to keep current behavior with error by default and 
just to recommend more prominently using file local header arguments

#+PROPERTY: header-args:sqlite :db ":memory"

Changing defaults is a compromise. Perhaps you mostly use src blocks as 
playground and in-memory database is really convenient for you. However 
missed :db file in a block creating tables may cause a *delayed* error 
that will happen on attempt to insert some data to these tables later. 
It is harder to figure out that :db parameter was forgotten several 
screens above. The cost of debugging might be higher than adding a 
property line to files where a user is experimenting with sqlite features.