[PATCH] sqlite: open database when we use it rather than at instantiation time
nicolas.s-dev at laposte.net
Fri May 13 13:09:52 BST 2016
On Fri, May 13, 2016 at 08:28:57AM +0200, Łukasz Żarnowiecki wrote:
> Thanks, what do you think about the idea of not closing connection to db
> when using autorefresh?
This would be yet another option. However, adding exceptions about how
we manage the db means supporting more than one context only for this.
I think this is prone to errors and might become yet another subtle bug
in the future.
> Opening and closing is quite expensive and
> maybe we could open at the beginning and close on some ctrl+c signal.
I don't think this is that expensive. I expect most of the work to be
done in RAM due to the kernel cache so this should be quite fast and
irrelevant. I usually request for numbers about that kind of claims to
avoid over-engineering. ,-)
More information about the OfflineIMAP-project